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.
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.