-
Notifications
You must be signed in to change notification settings - Fork 29
Release #311
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 Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. ✨ Finishing touches🧪 Generate unit tests (beta)
Tip 📝 Customizable high-level summaries are now available in beta!You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.
Example instruction:
Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later. 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 PR enhances the attribute annotation system for ReportPortal by making individual attribute annotations (@Attribute, @AttributeValue, @MultiKeyAttribute, @MultiValueAttribute) repeatable and usable standalone, in addition to being used within the @Attributes container annotation.
Key Changes:
- Individual attribute annotations can now be used directly on methods, types, and constructors (not just within
@Attributes) - Added support for repeatable annotations via new "Group" container annotations
- Introduced new
AttributeParser.retrieveAttributes(Executable)method to scan and extract attributes from methods and constructors - Refactored
AttributeParserwith overloaded methods for processing each annotation type
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
Attribute.java |
Made annotation repeatable, added target elements, and updated documentation |
AttributeValue.java |
Made annotation repeatable, added target elements, and updated documentation |
MultiKeyAttribute.java |
Made annotation repeatable, added target elements, and updated documentation |
MultiValueAttribute.java |
Made annotation repeatable, added target elements, and updated documentation |
AttributeGroup.java |
New container annotation for repeatable @Attribute |
AttributeValueGroup.java |
New container annotation for repeatable @AttributeValue |
MultiKeyAttributeGroup.java |
New container annotation for repeatable @MultiKeyAttribute |
MultiValueAttributeGroup.java |
New container annotation for repeatable @MultiValueAttribute |
Attributes.java |
Added ElementType.CONSTRUCTOR to target elements |
AttributeParser.java |
Added overloaded methods for each annotation type and new retrieveAttributes(Executable) method |
AnnotationAttributeParserTest.java |
Added comprehensive tests for standalone and mixed annotation usage scenarios |
CHANGELOG.md |
Documented the new features |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.