RC #315 - #316
Conversation
Reviewer's GuideRefreshes README to provide full CLI command reference and updates CI workflows to use Node 22/24 with npm 12 for Trusted Publishing compatibility, along with corresponding lockfile changes. Flow diagram for updated release pipeline CLI commandsflowchart TD
A[GitHub_Actions_workflow] --> B[auro_rc-workflow]
B --> C[RC_branch_and_PR_created]
A --> D[auro_pr-release]
D --> E[package_json_preview_version_set]
A --> F[auro_check-commits]
F --> G[PR_labels_and_release_notes]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
Fixed security issues:
-
brace-expansion (link)
-
fast-uri (link)
-
form-data (link)
-
immutable (link)
-
minimatch (link)
-
postcss (link)
-
svgo (link)
-
ws (link)
-
In the GitHub Actions workflows, consider upgrading npm before running
npm clean-installso that dependency installation also benefits from the trusted-publishing–compatible npm version instead of using two different npm versions in a single job. -
The
auro adoCLI option in the README (-g, --gh-issue <issue>) could clarify the expected format (issue number vs URL vsowner/repo#number) to prevent ambiguity for consumers.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In the GitHub Actions workflows, consider upgrading npm before running `npm clean-install` so that dependency installation also benefits from the trusted-publishing–compatible npm version instead of using two different npm versions in a single job.
- The `auro ado` CLI option in the README (`-g, --gh-issue <issue>`) could clarify the expected format (issue number vs URL vs `owner/repo#number`) to prevent ambiguity for consumers.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Alaska Airlines Pull Request
Release candidate pull request. See issue #315 for details.
Checklist:
By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team
Summary by Sourcery
Update Auro CLI documentation and align CI workflows with Node 22+/npm 12 requirements for trusted publishing.
Build:
CI:
Documentation: