-
Notifications
You must be signed in to change notification settings - Fork 1
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
AURO MIGRATION: Update Templates to v2 #55
Conversation
Reviewer's Guide by SourceryThis pull request updates the Auro templates to v2. It includes changes to the settings, pull request template, and issue templates. It also removes old workflows and issue templates. Flow diagram of updated GitHub issue templatesflowchart TD
A[Issue Creation] --> B{Select Template Type}
B -->|Bug Report| C[Bug Report Template]
B -->|Feature Request| D[Feature Request Template]
B -->|Support Request| E[Support Request Template]
B -->|Story| F[Story Template]
B -->|Task| G[Task Template]
B -->|Group| H[Group Template]
C --> I[Project Assignment<br/>AlaskaAirlines/19]
D --> I
E --> I
F --> I
G --> I
H --> I
subgraph Bug Report Details
C --> J[Browser Selection]
C --> K[Platform Selection]
C --> L[Reproduction Steps]
C --> M[Version Info]
end
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Surge demo deployment failed! 😭 |
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.
Hey @chrisfalaska - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
.github/PULL_REQUEST_TEMPLATE.md
Outdated
# Alaska Airlines Pull Request | ||
|
||
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. | ||
|
||
**Resolves:** # (issue, if applicable) | ||
|
||
## Summary: | ||
|
||
_Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project._ | ||
|
||
## Type of change: | ||
|
||
Please delete options that are not relevant. | ||
|
||
- [ ] New capability | ||
- [ ] Revision of an existing capability | ||
- [ ] Infrastructure change (automation, etc.) | ||
- [ ] Other (please elaborate) | ||
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for 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.
suggestion: Remove repeated sentence
This sentence is already present in the template before the changes. Removing the repetition will make the template cleaner.
# Alaska Airlines Pull Request | |
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. | |
**Resolves:** # (issue, if applicable) | |
## Summary: | |
_Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project._ | |
## Type of change: | |
Please delete options that are not relevant. | |
- [ ] New capability | |
- [ ] Revision of an existing capability | |
- [ ] Infrastructure change (automation, etc.) | |
- [ ] Other (please elaborate) | |
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. | |
# Alaska Airlines Pull Request | |
We identified an issue with the We will refine the Multi-Gitter process to prevent this in the future. As a result, the changes were committed, making this PR unnecessary. Closing PR. |
Resolves AlaskaAirlines/auro-cli#30
Summary by Sourcery
Update the required status checks for the
main
andbeta
branches to includetest (20.x)
andtest (22.x)
, and remove outdated issue templates, labels, and workflows.CI:
test (20.x)
andtest (22.x)
.Tests: