-
Notifications
You must be signed in to change notification settings - Fork 179
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
refactor: add pre-defined annotation keys into option.Packer #502
refactor: add pre-defined annotation keys into option.Packer #502
Conversation
Signed-off-by: Juncheng Zhu <junczhu@microsoft.com>
Signed-off-by: Juncheng Zhu <junczhu@microsoft.com>
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.
LGTM
Signed-off-by: Juncheng Zhu <junczhu@microsoft.com>
Signed-off-by: Juncheng Zhu <junczhu@microsoft.com>
Codecov Report
@@ Coverage Diff @@
## main #502 +/- ##
=======================================
Coverage 72.68% 72.68%
=======================================
Files 9 9
Lines 388 388
=======================================
Hits 282 282
Misses 85 85
Partials 21 21
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
LGTM
…oject#502) Signed-off-by: Juncheng Zhu <junczhu@microsoft.com>
Defined pre-defined annotation keys in
option.Packer
, and replaced other const defined with the same value.Updated annotation related example description.