-
-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat!: use packaging.Version
and other fixes and Ape-ifications
#161
Conversation
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
93e04b3
to
95ab54d
Compare
packaging.Version
and other updatespackaging.Version
and other fixes
packaging.Version
and other fixespackaging.Version
and other fixes and Ape-ifications
09463f2
to
09af34b
Compare
09af34b
to
803b409
Compare
permissions: | ||
contents: write |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad your adding this, opening a PR on the project-template now.
803b409
to
0cfc55b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
@@ -21,14 +21,18 @@ def solc_binary(): | |||
|
|||
|
|||
@pytest.fixture | |||
def nosolc(tmp_path, monkeypatch): | |||
def nosolc(tmp_path, mocker): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ya mocker is built on top of monkeypatch but i find its syntax easier to read in some cases.
Co-authored-by: NotPeopling2day <32708219+NotPeopling2day@users.noreply.github.com>
What I did
Major dust off of this package to try and get it moving again.
semantic_version
topackaging.Version
to properly handle RC versions.--help
command expected the wrong result code for some solc versionsfixes: #145
fixes: #147
How I did it
How to verify it
Checklist