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

Issue forms #562

Merged
merged 28 commits into from
Jun 27, 2023
Merged

Issue forms #562

merged 28 commits into from
Jun 27, 2023

Conversation

jhajduk-microsoft
Copy link
Contributor

First draft of Issue Forms for bugs, features, and general feedback

Replace this with a brief description of what this Pull Request fixes, changes, etc.

This PR fixes/adds/changes/removes

  1. Creation of the forms for bugs, features, and general feedback

Breaking Changes

N/A

Testing Evidence

Replace this with any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).

As part of this Pull Request I have

@ghost ghost added the Needs: Triage 🔍 Needs triaging by the team label Jun 20, 2023
@jtracey93 jtracey93 added hygiene and removed Needs: Triage 🔍 Needs triaging by the team labels Jun 22, 2023
Copy link
Collaborator

@jtracey93 jtracey93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @jhajduk-microsoft,

Thanks for the PR. Can you fix the linter errors please? https://github.com/Azure/ALZ-Bicep/actions/runs/5327851749/jobs/9651792603?pr=562#step:4:88

Should be as simple as opening each of the files in VSCode and Alt + Shift + F

make sure you have the editorconfig extension installed

Save each file and commit and push :)

@ghost ghost removed the Needs: Author Feedback label Jun 23, 2023
@jhajduk-microsoft
Copy link
Contributor Author

/azp run validateazcloud

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 562 in repo Azure/ALZ-Bicep

Copy link
Collaborator

@jtracey93 jtracey93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jhajduk-microsoft, do we need to remove the current issue templates as well in this PR?

@jhajduk-microsoft
Copy link
Contributor Author

@jtracey93 - Lets see how the new forms work in the repo and then I will submit a subsequent PR to remove the old forms

@ghost ghost removed the Needs: Author Feedback label Jun 26, 2023
@jtracey93 jtracey93 merged commit 3846bd7 into Azure:main Jun 27, 2023
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.

2 participants