Skip to content

Conversation

hardbyte
Copy link
Owner

No description provided.

@hardbyte
Copy link
Owner Author

Oh, travis previously published the wheels - I'll upload an a0 release manually but perhaps we should get github to build and publish the artifacts.

@hardbyte
Copy link
Owner Author

Scary how easy that is... https://pypi.org/project/python-can/4.1.0a0/

pip install python-can==4.1.0a0 
@hardbyte
Copy link
Owner Author

I've adjusted the coveralls settings to be a bit less picky:

image

@felixdivo
Copy link
Collaborator

Scary how easy that is

That's what I thought after my first upload (version 4.0.0) 😅 "Please don't upload something wrong ..."

But doing it in a fresh Docker container kinda helped to make sure to not be bothered with potentially misconfigured local setups.

@felixdivo felixdivo added this to the 4.1.0 Release milestone Nov 15, 2022
@zariiii9003
Copy link
Collaborator

We could setup automatic releases whenever a git tag starting with "v*" is pushed. Like this
That tag pattern must be added to protected tags in the project settings.

@felixdivo
Copy link
Collaborator

We could setup automatic releases whenever a git tag starting with "v*" is pushed.

This could also be used to more regularly push out pre-releases.

@zariiii9003
Copy link
Collaborator

Updating mypy might be a bit difficult, i tried before 😄
Let's leave that for a separate PR.

@hardbyte
Copy link
Owner Author

hardbyte commented Nov 16, 2022

Updating mypy might be a bit difficult, i tried before smile Let's leave that for a separate PR.

image

🤞🏼 it works for everything on the CI though...

@zariiii9003
Copy link
Collaborator

🤞🏼 it works for everything on the CI though...

I had trouble getting it to pass on both Windows and Linux because of "warn_unused_ignores = True". It was always either Windows OR Linux, not both

@hardbyte
Copy link
Owner Author

I had trouble getting it to pass on both Windows and Linux because of "warn_unused_ignores = True". It was always either Windows OR Linux, not both

As we're only running the static analysis tests in linux I think that we're okay with mypi.

I am wondering why pylint is passing fine locally but not on the CI though. Do we have a circular import problem?

************* Module examples.virtual_can_demo examples/virtual_can_demo.py:1:0: R0401: Cyclic import (can -> can.notifier -> can.bus) (cyclic-import) examples/virtual_can_demo.py:1:0: R0401: Cyclic import (can -> can.interface -> can.util) (cyclic-import) examples/virtual_can_demo.py:1:0: R0401: Cyclic import (can -> can.util) (cyclic-import) examples/virtual_can_demo.py:1:0: R0401: Cyclic import (can -> can.notifier -> can.listener -> can.bus) (cyclic-import) examples/virtual_can_demo.py:1:0: R0401: Cyclic import (can -> can.interface -> can.bus) (cyclic-import) examples/virtual_can_demo.py:1:0: R0401: Cyclic import (can -> can.thread_safe_bus -> can.interface -> can.util) (cyclic-import) 

In any case, I'm going to call it for tonight. Feel free to push to this branch (even reverting the pylint update).

@hardbyte
Copy link
Owner Author

Ok I lied, I'm waiting to see that green tick...

@hardbyte
Copy link
Owner Author

Prerelease uploaded by Github action (after I created the release in github UI)

https://pypi.org/project/python-can/4.1.0a1/

@hardbyte hardbyte marked this pull request as ready for review November 16, 2022 11:16
@hardbyte hardbyte requested review from felixdivo and zariiii9003 and removed request for felixdivo November 16, 2022 11:16
@hardbyte hardbyte requested a review from felixdivo November 16, 2022 11:16
@hardbyte hardbyte mentioned this pull request Nov 16, 2022
Copy link
Collaborator

@felixdivo felixdivo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for preparing the release 🙂

Copy link
Collaborator

@zariiii9003 zariiii9003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@j-c-cook
Copy link
Contributor

🚀

@pierreluctg
Copy link
Collaborator

👍

@hardbyte
Copy link
Owner Author

Found a bug while testing examples - appreciate a second pair of eyes - #1440

hardbyte and others added 2 commits November 18, 2022 19:28
* Allow restarting of transmission tasks for socketcan * Update can/interfaces/socketcan/socketcan.py Co-authored-by: zariiii9003 <52598363+zariiii9003@users.noreply.github.com>
@zariiii9003
Copy link
Collaborator

@hardbyte Could you try to remove appveyor?
image
We migrated all Windows tests to github anyway.

@hardbyte
Copy link
Owner Author

I've just removed the app veyor hook

@felixdivo
Copy link
Collaborator

Good catch in #1440. Is it in the changelog yet?

@hardbyte
Copy link
Owner Author

Good catch in #1440. Is it in the changelog yet?

Thanks! Yes I've just pushed it.

@hardbyte hardbyte merged commit 8816061 into develop Nov 24, 2022
@mergify mergify bot deleted the release-4.1.0 branch November 24, 2022 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants