-
Notifications
You must be signed in to change notification settings - Fork 1
refactor(release): refactoring release process a bit. #3
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
Conversation
Add issue templates Add release drafter Remove version files
Merging this branch will decrease overall coverage
Coverage by fileChanged files (no unit tests)
Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code. Changed unit test files
|
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.
Pull Request Overview
Refactors the release process by removing the existing version files, adding a Release Drafter workflow, and introducing GitHub issue templates.
- Remove hardcoded version constant and its test
- Add automated Release Drafter workflow and configuration
- Provide standardized feature request and bug report templates
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
version_test.go | Deleted test for Version() |
version.go | Deleted version constant and Version() function |
.github/workflows/release-drafter.yml | New workflow file to auto-draft releases |
.github/release-drafter.yml | Configuration for the Release Drafter action |
.github/ISSUE_TEMPLATE/feature_request.md | New feature request issue template |
.github/ISSUE_TEMPLATE/config.yml | Settings to disable blank issues and link docs/repo |
.github/ISSUE_TEMPLATE/bug_report.md | New bug report issue template |
Comments suppressed due to low confidence (2)
version_test.go:1
- Removing the version test eliminates coverage for version functionality; ensure the new release process includes tests or integration checks for version resolution.
-package entraid
version.go:1
- The
Version()
API and its backing constant have been removed; update any callers or provide migration guidance to prevent breaking changes.
-package entraid
@@ -0,0 +1,26 @@ | |||
name: Release Drafter |
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.
[nitpick] Consider adding a brief comment or linking to higher-level docs explaining how this workflow fits into the overall release and versioning process.
Copilot uses AI. Check for mistakes.
@@ -0,0 +1,46 @@ | |||
name-template: 'v$RESOLVED_VERSION' |
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.
[nitpick] Having two release-drafter configuration files in different directories may be confusing; consider consolidating them or documenting their distinct roles.
Copilot uses AI. Check for mistakes.
Add issue templates
Add release drafter
Remove version files