Merged
Conversation
jasonsaayman
added
priority::medium
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors the CI/CD infrastructure for the v0.x branch of axios, consolidating workflows and updating them to use modern GitHub Actions. The changes streamline the release process with manual publishing controls, expand Node.js version coverage in CI, and fix a flaky test.
Changes:
- Introduces a new unified CI workflow (run-ci.yml) testing Node 12-24 with build/test steps and dependency review
- Replaces the publish workflow with a manual trigger supporting patch/minor releases and optional beta versions
- Adds Husky commit-msg hook for commitlint validation, locks dtslint version to prevent drift, and fixes an HTTP adapter test to use ephemeral ports
- Removes legacy workflows (ci.yml, codeql-analysis.yml, depsreview.yaml, npm-tag.yml, stale.yml) and consolidates issue templates
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/run-ci.yml | New unified CI workflow with Node 12-24 matrix testing and dependency review |
| .github/workflows/publish.yml | Refactored manual publish workflow with version bump automation |
| .github/workflows/ci.yml | Removed legacy CI workflow |
| .github/workflows/codeql-analysis.yml | Removed CodeQL workflow |
| .github/workflows/depsreview.yaml | Removed standalone dependency review (now in run-ci.yml) |
| .github/workflows/npm-tag.yml | Removed manual NPM tagging workflow |
| .github/workflows/stale.yml | Removed stale issue/PR management workflow |
| .husky/commit-msg | Added Husky hook to run commitlint on commit messages |
| package.json | Locked dtslint to exact version 4.2.1 |
| test/unit/adapters/http.js | Fixed flaky test by using ephemeral port (0) and removed brittle error message assertion |
| .github/ISSUE_TEMPLATE.md | Updated branch references from master to main, improved environment fields |
| .github/ISSUE_TEMPLATE/*.md | Removed individual issue templates in favor of consolidated template |
| .github/PULL_REQUEST_TEMPLATE.md | Updated contributing guide link from master to main branch |
Co-authored-by: Copilot <[email protected]>
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.
Summary by cubic
Refreshes CI and release workflows for v0.x with a single, faster pipeline and a manual publish flow that supports patch/minor and optional beta pre-releases. Adds a small fix to publish.yml to ensure version bumps target v0.x and publish with the correct token.
Description
Docs
Testing
Written for commit 41a0069. Summary will update on new commits.