-
Notifications
You must be signed in to change notification settings - Fork 34
Release #215
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
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Pull Request Overview
This release updates the library to version 5.6.5, adding OAuth 2.0 Password Grant authentication support and improving configuration parameter handling.
- OAuth 2.0 authentication mechanism with 6 new configuration parameters
- Deprecated
RP_LOG_BATCH_PAYLOAD_SIZEin favor ofRP_LOG_BATCH_PAYLOAD_LIMIT - Removed deprecated
RP_UUIDparameter support and updated authentication validation logic
Reviewed Changes
Copilot reviewed 10 out of 12 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| setup.py | Version bump to 5.6.5 |
| requirements.txt | Updated reportportal-client dependency to ~=5.6.7 |
| requirements-dev.txt | Added black and isort for code formatting |
| robotframework_reportportal/variables.py | Added OAuth 2.0 parameters, renamed log_batch_payload_size to log_batch_payload_limit, removed RP_UUID support |
| robotframework_reportportal/service.py | Updated client initialization with new parameter names and OAuth 2.0 fields |
| robotframework_reportportal/listener.py | Added error handling for service initialization with proper warning |
| tests/unit/conftest.py | Updated mock_variables fixture to use empty list for launch_attributes and removed unused imports |
| tests/integration/test_variables.py | Updated tests for new parameter names, removed deprecated tests, improved test structure |
| tests/init.py | Removed unused REQUESTS_SERVICE constant |
| README.md | Updated documentation with OAuth 2.0 configuration examples and renamed parameter |
| CHANGELOG.md | Documented changes for version 5.6.5 |
| .gitignore | Added Python virtual environment directories |
Comments suppressed due to low confidence (2)
README.md:2
- Line 0 appears in the diff, which suggests an ordering or numbering issue. The
RP_API_KEYline should be removed from the 'Required' section since it's now shown separately as one of two authentication options below.
README.md:2 - Line 0 with the deprecated
RP_LOG_BATCH_PAYLOAD_SIZEparameter appears in the diff. This deprecated parameter should be removed from the documentation or clearly marked as deprecated with a note to useRP_LOG_BATCH_PAYLOAD_LIMITinstead.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.