Closed reactive-firewall closed 4 days ago
The pull request introduces updates to the tests/requirements.txt
and requirements.txt
files, focusing on the license information for various dependencies. It standardizes the spelling of "license," clarifies previously ambiguous licenses, and comments out some packages that are no longer active dependencies. These changes enhance the documentation regarding licensing without affecting the functionality of the code.
File | Change Summary |
---|---|
tests/requirements.txt | Updated license comments for multiple dependencies, changing "licence" to "license" and clarifying license types for pep8 , pytest , and others. Commented out inactive dependencies. |
requirements.txt | Standardized "license" spelling and specified licenses for sphinx , setuptools , pip , build , multicast , and wheel . Commented out inactive dependencies. |
tests/requirements.txt
file, directly related to licensing information modifications.wheel
package in tests/requirements.txt
, aligning with the main PR changes.tests/requirements.txt
.Documentation
🐇 In the garden of code, where the licenses play,
A hop here, a skip there, we clarify today.
From PSF to MIT, the truth we now share,
With a flick of the tail, we spread knowledge with care.
So let’s dance in the fields, with comments so bright,
For every small change makes our project just right! 🌼
Tests completed | Failed | Passed | Skipped |
---|---|---|---|
414 | 3 | 411 | 6 |
tests.context tests.context.checkCovCommand
Stack Traces | 0.004s run time
> > ``` > 259 args (list): A list of command arguments, defaulting to [None]. > 260 > 261 Returns: > 262 list: The modified list of arguments with 'coverage run' options added as applicable. > 263 > 264 Examples: > 265 >>> checkCovCommand(["python", "script.py"]) > 266 ['python', 'script.py'] > 267 > 268 >>> checkCovCommand(["coverage", "script.py"]) # missing 'run' > Expected: > ['coverage', 'run', '-p', '--context=Integration', '--source=multicast', 'script.py'] > Got: > ['exit 1 ; #', 'run', '-p', '--context=Integration', '--source=multicast', 'script.py'] > > .../multicast/tests/context.py:268: DocTestFailure > ```tests.context tests.context.checkCovCommand
Stack Traces | 0.004s run time
> > ``` > 259 args (list): A list of command arguments, defaulting to [None]. > 260 > 261 Returns: > 262 list: The modified list of arguments with 'coverage run' options added as applicable. > 263 > 264 Examples: > 265 >>> checkCovCommand(["python", "script.py"]) > 266 ['python', 'script.py'] > 267 > 268 >>> checkCovCommand(["coverage", "script.py"]) # missing 'run' > Expected: > ['coverage', 'run', '-p', '--context=Integration', '--source=multicast', 'script.py'] > Got: > ['exit 1 ; #', 'run', '-p', '--context=Integration', '--source=multicast', 'script.py'] > > .../multicast/tests/context.py:268: DocTestFailure > ```
To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard
@reactive-firewall: No problem! I'll continue reviewing the PR now.
Summary by CodeRabbit
virtualenv
,six
,pgpy
, andtox
.