-
Notifications
You must be signed in to change notification settings - Fork 317
fix(instructions-janitor): use GitHub MCP get_latest_release instead of git describe #23625
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -37,7 +37,6 @@ tools: | |||||||||||||||||
| - "cat .github/aw/github-agentic-workflows.md" | ||||||||||||||||||
| - "wc -l .github/aw/github-agentic-workflows.md" | ||||||||||||||||||
| - "git log --since='*' --pretty=format:'%h %s' -- docs/" | ||||||||||||||||||
| - "git describe --tags --abbrev=0" | ||||||||||||||||||
|
|
||||||||||||||||||
| timeout-minutes: 15 | ||||||||||||||||||
|
|
||||||||||||||||||
|
|
@@ -55,13 +54,13 @@ Analyze documentation changes since the latest release and ensure the instructio | |||||||||||||||||
|
|
||||||||||||||||||
| ### 1. Identify Latest Release | ||||||||||||||||||
|
|
||||||||||||||||||
| Determine the latest release version to establish a baseline: | ||||||||||||||||||
| Determine the latest release version and its publish date: | ||||||||||||||||||
|
|
||||||||||||||||||
| ```bash | ||||||||||||||||||
| git describe --tags --abbrev=0 | ||||||||||||||||||
| ``` | ||||||||||||||||||
| get_latest_release(owner="github", repo="gh-aw") | ||||||||||||||||||
| ``` | ||||||||||||||||||
|
|
||||||||||||||||||
| If no tags exist, use the date from the CHANGELOG.md file to find the latest release version. | ||||||||||||||||||
| Use the `tag_name` field as the release version and the `published_at` field as `RELEASE_DATE`. | ||||||||||||||||||
|
|
||||||||||||||||||
|
||||||||||||||||||
| If `get_latest_release` returns no result (for example, the repository has no releases and the API returns 404), fall back to a deterministic baseline: | |
| - Use the GitHub toolset to list tags and select the most recent tag: | |
| - `list_tags(owner="github", repo="gh-aw")` (or equivalent). | |
| - Use the latest tag’s `name` as the release version. | |
| - Set `RELEASE_DATE` to the commit date of that tag (for example, via `git log -1 --format="%aI" TAG_NAME` or the corresponding commit date from the GitHub API). |
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.
The code fence for the
get_latest_release(...)call is missing a language tag (e.g.,text). This file’s own style guidelines (see “Use YAML/markdown code blocks with appropriate language tags”) and other workflow instruction files use language tags, so this block should be updated for consistency/readability.