Skip to content
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

Update issue-reporter.ts #5983

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update issue-reporter.ts #5983

wants to merge 1 commit into from

Conversation

Kynlos
Copy link
Collaborator

@Kynlos Kynlos commented Oct 23, 2024

Updated the wording and changed the order of the items to make it clearer to the user that they need to fill in the form.

This will hopefully cut down on reports that contain absolutely no information on GitHub Issues.

Test plan

Changelog

Updated the wording and changed the order of the items to make it clearer to the user that they need to fill in the form.

This will hopefully cut down on reports that contain absolutely no information on GitHub Issues.
jamesmcnamara added a commit that referenced this pull request Oct 23, 2024
Hoping to unblock #5983 and #5982.

In #4923 we added the google
auth action to the test-unit step but did not exclude the step if the PR
comes from a fork (as we do with the other auth steps).

@akalia25 Do you see any issue with this tact? Should the setup-gcloud
step get skipped too?

## Test plan
Runs in CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant