Conversation
…-Dependabot' of https://github.com/blindzero/IdentityLifecycleEngine into issues/15-Repo-hygiene-with-issue-and-PR-templates-with-Dependabot
…-Dependabot' of https://github.com/blindzero/IdentityLifecycleEngine into issues/15-Repo-hygiene-with-issue-and-PR-templates-with-Dependabot
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e63072c8f2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
This PR improves GitHub issue and PR templates by simplifying automated workflows and standardizing template configurations. The workflow that auto-assigns issues is effectively disabled since issue templates now handle assignment and labeling directly.
Changes:
- Changed workflow trigger from automatic (on issue open) to manual dispatch
- Reorganized PR template structure, moving "Changes" section after "Type of Change"
- Updated issue template frontmatter to use multi-line YAML format and add direct assignment/labels
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| .github/workflows/issue-auto-assign.yml | Changed from automatic to manual trigger (workflow_dispatch) |
| .github/PULL_REQUEST_TEMPLATE/pull_request_template.md | Reordered sections and added "How to test & review" subsection |
| .github/ISSUE_TEMPLATE/feature_request.md | Removed title prefix, added assignee and labels directly in template |
| .github/ISSUE_TEMPLATE/bug_report.md | Removed title prefix, added assignee and labels directly in template |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…-Dependabot' of https://github.com/blindzero/IdentityLifecycleEngine into issues/15-Repo-hygiene-with-issue-and-PR-templates-with-Dependabot
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Comments suppressed due to low confidence (1)
.github/PULL_REQUEST_TEMPLATE/pull_request_template.md:1
- The PR template is in the wrong location. GitHub expects the pull request template to be at '.github/pull_request_template.md' (or '.github/PULL_REQUEST_TEMPLATE.md'), not '.github/PULL_REQUEST_TEMPLATE/pull_request_template.md'. The template file should be moved to the correct location for GitHub to recognize it automatically.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Comments suppressed due to low confidence (1)
.github/PULL_REQUEST_TEMPLATE/pull_request_template.md:1
- The PR template file is incorrectly located. GitHub expects PR templates to be at '.github/pull_request_template.md' (singular, no TEMPLATE folder) or '.github/PULL_REQUEST_TEMPLATE.md'. The current location '.github/PULL_REQUEST_TEMPLATE/pull_request_template.md' (with both a folder and 'template' in the filename) is redundant and may not be recognized by GitHub. Move the file to '.github/pull_request_template.md' or '.github/PULL_REQUEST_TEMPLATE.md'.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| > | ||
| > - Bug report: https://github.com/blindzero/IdentityLifecycleEngine/issues/new?template=bug_report.md | ||
| > - Feature request: https://github.com/blindzero/IdentityLifecycleEngine/issues/new?template=feature_request.md | ||
| --- |
There was a problem hiding this comment.
There are two consecutive horizontal rules (---) on lines 11 and 16 without content between them, creating an empty blockquote section. This appears to be a formatting error. Remove one of the horizontal rules or add content between them.
| --- |
Summary
Improved issue templates and PR template with simplified actions (not necessary anymore).
Changes
newto labelsasCloses #15