You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Feel free to do your own searches to find more examples if you want, then change our code to use keyword arguments, which should hopefully make the code more readable overall as well as removing the warnings.
To reproduce
Run pytest on test/test_available_fix.py
Expected behaviour: no warnings
Actual behaviour: the warning above appears
cve-bin-tool uses https://www.conventionalcommits.org/ style for commit messages, and we have a test that checks the title of your pull request (PR). A good potential title for this one is in the title of this issue.
You can make an issue auto close by including a comment "fixes #ISSUENUMBER" in your PR comments where ISSUENUMBER is the actual number of the issue. This "links" the issue to the pull request.
Claiming issues:
You do not need to have an issue assigned to you before you work on it. To "claim" an issue either make a linked pull request or comment on the issue saying you'll be working on it.
If someone else has already commented or opened a pull request, assume it is claimed and find another issue to work on.
If it's been more than 1 week without progress, you can ask in a comment if the claimant is still working on it before claiming it yourself (give them at least 3 days to respond before assuming they have moved on).
The text was updated successfully, but these errors were encountered:
Description
We're currently getting the following warning:
Here's the python docs on positional vs keyword arguments:
https://docs.python.org/3/tutorial/controlflow.html#special-parameters
Feel free to do your own searches to find more examples if you want, then change our code to use keyword arguments, which should hopefully make the code more readable overall as well as removing the warnings.
To reproduce
Expected behaviour: no warnings
Actual behaviour: the warning above appears
Anything else?
Short tips for new contributors:
Claiming issues:
The text was updated successfully, but these errors were encountered: