-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat(forge): combine all scripts into one #29
Conversation
8bfc77c
to
34e9750
Compare
83d9ba4
to
ed5075b
Compare
This change creates a forge script that merges scattered functionality - list packages - find packages - forge compile - forge bump - forge release
ed5075b
to
ff3056a
Compare
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
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.
PR Overview
This PR consolidates multiple package management scripts into a single forge script that handles compiling, bumping, and releasing packages. Key changes include:
- Adding new commands for compiling, bumping, and releasing packages via the forge CLI.
- Consolidating and refactoring existing scripts in the tool/forge directory.
- Updating GitHub workflows and example app entry points.
Reviewed Changes
File | Description |
---|---|
tool/forge/cli.ts | Introduces new forge commands with options and command chaining for package management. |
tool/forge/bump.ts | Implements version bumping and pull request creation for package updates. |
app/example/index.ts | Provides a new entry point with version logging. |
tool/forge/changelog.ts | Adds a changelog function to generate package update summaries. |
tool/forge/release.ts | Refactors release functionality and updates release asset uploads. |
tool/forge/compile.ts | Refactors the compile command and targets function, and improves installation option handling. |
.github/workflows/bump.yml | Updates bump workflow to use the new forge bump command with pull request creation. |
.github/workflows/ci.yml | Adjusts CI workflow to call the new forge compile command. |
core/package/package.ts | Revises workspace and package info functions with updated documentation and directory roots. |
.github/workflows/release.yml | Updates release workflow to use the new forge release command with the draft option. |
app/example/example.ts | Removes the old example file in favor of the new index entry point. |
Copilot reviewed 14 out of 14 changed files in this pull request and generated 3 comments.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
This change creates a forge script that merges scattered functionality.
> forge 📦 app/example 📦 core/async @roka/async 0.2.0-pre.1+5386c7c 📦 core/cli @roka/cli 0.2.0-pre.2+83d6b52 📦 core/git @roka/git 0.2.0-pre.8+bc8fe99 📦 core/github @roka/github 0.1.1-pre.1+d3112dc 📦 core/http @roka/http 0.2.0-pre.1+34a4db5 📦 core/package @roka/package 0.0.2-pre.1+ee7617b 📦 core/testing @roka/testing 0.2.0-pre.12+13ba1d1 📦 tool/forge @roka/forge 0.1.0-pre.1+ff3056a
> forge cli 📦 core/cli @roka/cli 0.2.0-pre.2+83d6b52
> forge compile example 📦 Compiled example 🏺 dist/example/aarch64-apple-darwin/example
> forge bump --pr 📦 Bumped package versions 🚀 Created version bump pull request [https://github.com/withroka/roka/pull/27]
> forge release cli --draft 🚀 Released cli [https://github.com/withroka/roka/releases/tag/untagged-53929bdda258723cf9de]