Commit 5df9ede
authored
fix: fix artifact paths in publish and publish-npm jobs (#2635)
## Summary
Fixes two broken CI jobs that prevented releases from completing.
## Failures
### 1. `publish` job — `ls: cannot access 'out/*.vsix'`
`download-artifact` without `path:` strips the `out/` prefix, so vsix
files landed in workspace root instead of `out/`. Both `ls -la
out/*.vsix`
and `./tools/helper --publish` (which globs `out/ansible-*.vsix`)
failed.
Also fixes `files: "*.vsix"` → `files: "out/*.vsix"` in the GitHub
release
attachment step for consistency.
### 2. `publish-npm` job — Artifact not found
The job requested artifact
`@ansible-ansible-language-server-build-<id>.tgz`
which was never uploaded. The `.tgz` is bundled inside the main artifact
`ansible-extension-build-<id>.zip`. Fixed to download the correct
artifact.
### 3. `publish-npm` job — `.github/actions/report` not found
Missing checkout step caused the local report action to be unresolvable.
Added `actions/checkout` before the report step.
## Changes
- `publish`: add `path: out` to download step
- `publish`: `files: "*.vsix"` → `files: "out/*.vsix"`
- `publish-npm`: fix artifact name to use the main build artifact
- `publish-npm`: add `path: out` to download step
- `publish-npm`: fix npm publish glob to
`out/@ansible-ansible-language-server-*.tgz`
- `publish-npm`: add checkout step so `.github/actions/report` resolves1 parent e6b9035 commit 5df9ede
1 file changed
Lines changed: 8 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
587 | 587 | | |
588 | 588 | | |
589 | 589 | | |
| 590 | + | |
590 | 591 | | |
591 | 592 | | |
592 | 593 | | |
593 | 594 | | |
594 | 595 | | |
595 | 596 | | |
596 | | - | |
| 597 | + | |
597 | 598 | | |
598 | 599 | | |
599 | 600 | | |
| |||
624 | 625 | | |
625 | 626 | | |
626 | 627 | | |
| 628 | + | |
| 629 | + | |
| 630 | + | |
627 | 631 | | |
628 | 632 | | |
629 | 633 | | |
630 | | - | |
| 634 | + | |
| 635 | + | |
631 | 636 | | |
632 | 637 | | |
633 | 638 | | |
634 | | - | |
| 639 | + | |
635 | 640 | | |
636 | 641 | | |
637 | 642 | | |
| |||
0 commit comments