-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fix: bug.yml #4621
fix: bug.yml #4621
Conversation
@@ -27,16 +27,16 @@ body: | |||
attributes: | |||
label: Which package are you using? | |||
options: | |||
- @sentry/angular |
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.
Why this change?
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.
It can't compile yml when there is @
in front of it
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.
Then let’s add quotes to force it to become a string instead of making yaml coerce it?
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.
That actually works. Just tested it.
size-limit report
|
@@ -57,7 +57,7 @@ body: | |||
id: link-to-sentry | |||
attributes: | |||
label: Link to Sentry event | |||
description: If applicable, provide a link to the affected event from your Sentry account. The event will only be viewable by Sentry staff. **Note**: This should be an _event_ link, not an issue link, and should match the form shown below. It can be found by clicking on the event ID on the issue details page. | |||
description: If applicable, provide a link to the affected event from your Sentry account. The event will only be viewable by Sentry staff. |
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.
Why remove this? (It's something even our staff get wrong without periodic reminders.)
Before submitting a pull request, please take a look at our
Contributing guidelines and verify:
yarn lint
) & (yarn test
).