-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add tagging func to a resource for a specific revision #1425
Conversation
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.
Code-wise good to me - I've left two comments that are not mandatory. I'll test it now.
}, | ||
{ | ||
pattern: /^[a-zA-Z0-9_-]+$/, | ||
message: 'Tag must contains only letters and numbers', |
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.
Two things:
-
Just double checking, are dashes really allowed by delta? I thought it was only alpha-numeric characters or underscore. I may be misrembering though.
-
The message says that only letters and numbers are allowed. Maybe we should also add that dashes and underscores are also allowed.
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.
- Simon has mentioned that the orgs name and tags should have the same regex (may be we forget to mention the _ and -)
- you're right message updated.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #1425 +/- ##
===========================================
- Coverage 49.65% 47.99% -1.66%
===========================================
Files 249 249
Lines 11339 11362 +23
Branches 2652 2655 +3
===========================================
- Hits 5630 5453 -177
- Misses 5673 5876 +203
+ Partials 36 33 -3
☔ View full report in Codecov by Sentry. |
Allow the user to tag a resource for a specific revision
Description
How has this been tested?
Types of changes
Checklist: