fix: ghaw compat test apm.yml uses correct manifest format#363
Merged
danielmeppiel merged 1 commit intomainfrom Mar 18, 2026
Merged
fix: ghaw compat test apm.yml uses correct manifest format#363danielmeppiel merged 1 commit intomainfrom
danielmeppiel merged 1 commit intomainfrom
Conversation
Match apm-action's generateManifest() format: name, version, and dependencies under 'dependencies.apm' key (not top-level array). Verified locally: tokenless install + pack succeeds. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the GH-AW compatibility portion of the release validation script so the generated apm.yml matches the manifest shape produced by apm-action (and APM’s documented manifest schema), ensuring the tokenless install + pack flow is exercised with a valid manifest.
Changes:
- Add required
versionfield to the generatedapm.ymlin the GH-AW compat test. - Fix
dependenciesto the correct object shape (dependencies.apmlist) instead of a top-level list. - Adjust the inline comment to reflect that the manifest matches
apm-action’sgenerateManifest()format.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The test apm.yml was missing
versionand used the wrong dependency structure. Fixed to matchapm-action'sgenerateManifest()format:Verified locally: tokenless install + pack succeeds with this format.