-
Notifications
You must be signed in to change notification settings - Fork 2
v2 #13
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
base: v1
Are you sure you want to change the base?
v2 #13
Changes from all commits
f5bb657
9b0d6d2
7a6d1bd
c433d00
9fc3783
89aff6a
4ee169b
77ca2c5
2211332
ab3f5b6
666245f
f9a3a34
dbdea85
c2d7e12
b2db7ae
1beed56
ac25caa
122abeb
4463309
26ca899
a3b4032
6876bc7
33794b5
f4bf6fc
acc2628
f32e7a0
002cbcd
9532c05
78a7d6a
d0dfa30
9128260
88d8c61
c477e53
5d30d63
7c58e3f
d1d1a18
cf2f57f
4617fa1
5a1fdec
1e57575
650215b
5d825ce
6d5d8b0
6e0c220
5155051
1d0752e
b6cd0b2
f8ea2bf
861bb2f
2348f5e
58e787a
026e022
3ed8780
dc78fcc
12bb493
8c74a12
47d1072
34952d7
c0ac804
7c9830f
2e7557c
787e462
b5bbc33
aa52d79
a893bf6
b880477
5096bec
1c5a984
054dc9c
96ac400
91a192e
8b6dbc2
c0e75df
35468ca
020101a
e3d30ae
3010605
c1cede8
23cd242
fc42d4b
1f82320
46c509d
41c679f
571d69f
168b81b
a291139
26dc5f8
a1fcdce
54c4c32
6f6b38b
2bdf366
d6edc23
ef32a5c
845dccd
fb33b91
a74d6b4
4ad5310
08dcc27
b5933e3
e5f7f3d
3f8d802
b97ef58
89478d8
1759ffc
4495124
2f8701b
545ea53
3602133
4055c44
4319760
5de7c7f
323ddc3
49917db
017af9b
039a409
2926365
a7761af
cef7c79
cd05550
dbf52fe
8e9776c
2988ac3
d755060
eb6cb16
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| # Changesets | ||
|
|
||
| Hello and welcome! This folder has been automatically generated by `@changesets/cli`, a build tool that works | ||
| with multi-package repos, or single-package repos to help you version and publish your code. You can | ||
| find the full documentation for it [in our repository](https://github.com/changesets/changesets) | ||
|
|
||
| We have a quick list of common questions to get you started engaging with this project in | ||
| [our documentation](https://github.com/changesets/changesets/blob/main/docs/common-questions.md) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| { | ||
| "$schema": "https://unpkg.com/@changesets/config@3.1.1/schema.json", | ||
| "changelog": "@changesets/cli/changelog", | ||
| "commit": false, | ||
| "fixed": [], | ||
| "linked": [["@blac/*"]], | ||
| "access": "public", | ||
| "baseBranch": "v1", | ||
| "updateInternalDependencies": "patch", | ||
| "ignore": ["demo", "blac-docs", "perf"] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| --- | ||
| '@blac/plugin-render-logging': patch | ||
| '@blac/plugin-persistence': patch | ||
| '@blac/react': patch | ||
| '@blac/core': patch | ||
| --- | ||
|
|
||
| build to js |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| { | ||
| "mode": "pre", | ||
| "tag": "rc", | ||
| "initialVersions": { | ||
| "@blac/core": "2.0.0-rc-16", | ||
| "@blac/react": "2.0.0-rc-16", | ||
| "@blac/plugin-persistence": "2.0.0-rc-16", | ||
| "@blac/plugin-render-logging": "2.0.0-rc-16", | ||
| "demo": "1.0.0", | ||
| "blac-docs": "1.0.0", | ||
| "perf": "1.0.0" | ||
| }, | ||
| "changesets": [ | ||
| "cyan-rats-vanish" | ||
| ] | ||
| } |
This file was deleted.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,50 @@ | ||
| name: CI | ||
|
|
||
| on: | ||
| push: | ||
| branches: [main, v1] | ||
| pull_request: | ||
| branches: [main, v1] | ||
|
|
||
| jobs: | ||
| build-and-test: | ||
| name: Build and Test | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| node-version: [22.x] | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Install pnpm | ||
| uses: pnpm/action-setup@v4 | ||
| with: | ||
| version: 10.14.0 | ||
|
|
||
| - name: Use Node.js ${{ matrix.node-version }} | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: ${{ matrix.node-version }} | ||
| cache: 'pnpm' | ||
|
|
||
| - name: Install dependencies | ||
| run: pnpm install --frozen-lockfile | ||
|
|
||
| - name: Build packages | ||
| run: pnpm build | ||
|
|
||
| - name: Run tests | ||
| run: pnpm test | ||
|
|
||
| - name: Run type checks | ||
| run: pnpm typecheck | ||
|
|
||
| - name: Run linter | ||
| run: pnpm lint | ||
|
|
||
| - name: Check formatting | ||
| run: pnpm format --check |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,69 @@ | ||
| name: Manual Release | ||
|
|
||
| on: | ||
| workflow_dispatch: | ||
| inputs: | ||
| tag: | ||
| description: 'NPM tag to publish to (e.g., latest, preview, beta)' | ||
| required: true | ||
| default: 'preview' | ||
| type: choice | ||
| options: | ||
| - latest | ||
| - preview | ||
| - beta | ||
| - alpha | ||
|
|
||
| permissions: | ||
| contents: write | ||
| id-token: write | ||
|
|
||
| jobs: | ||
| release: | ||
| name: Manual Release | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Install pnpm | ||
| uses: pnpm/action-setup@v4 | ||
| with: | ||
| version: 10.14.0 | ||
|
|
||
| - name: Setup Node.js | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 22 | ||
| cache: 'pnpm' | ||
| registry-url: 'https://registry.npmjs.org' | ||
|
|
||
| - name: Install dependencies | ||
| run: pnpm install --frozen-lockfile | ||
|
|
||
| - name: Build packages | ||
| run: pnpm build | ||
|
|
||
| - name: Run tests | ||
| run: pnpm test | ||
|
|
||
| - name: Run type checks | ||
| run: pnpm typecheck | ||
|
|
||
| - name: Publish to npm | ||
| run: | | ||
| pnpm changeset publish --tag ${{ inputs.tag }} | ||
| env: | ||
| NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
|
|
||
| - name: Create GitHub Release | ||
| uses: actions/create-release@v1 | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| with: | ||
| tag_name: v${{ steps.version.outputs.version }} | ||
| release_name: Release v${{ steps.version.outputs.version }} | ||
| draft: false | ||
| prerelease: ${{ inputs.tag != 'latest' }} | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,59 @@ | ||
| name: Release | ||
|
|
||
| on: | ||
| push: | ||
| branches: | ||
| - v1 | ||
|
|
||
| permissions: | ||
| contents: write | ||
| pull-requests: write | ||
| id-token: write | ||
|
|
||
| jobs: | ||
| release: | ||
| name: Release | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
|
Comment on lines
+13
to
+17
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Add a If multiple commits land on 🤖 Prompt for AI Agents |
||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Install pnpm | ||
| uses: pnpm/action-setup@v4 | ||
| with: | ||
| version: 10.14.0 | ||
|
|
||
| - name: Setup Node.js | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 22 | ||
| cache: 'pnpm' | ||
| registry-url: 'https://registry.npmjs.org' | ||
|
|
||
| - name: Install dependencies | ||
| run: pnpm install --frozen-lockfile | ||
|
|
||
| - name: Build packages | ||
| run: pnpm build | ||
|
|
||
| - name: Run tests | ||
| run: pnpm test | ||
|
|
||
| - name: Create Release Pull Request or Publish | ||
| id: changesets | ||
| uses: changesets/action@v1 | ||
| with: | ||
| publish: pnpm release | ||
| title: 'chore: release packages' | ||
| commit: 'chore: release packages' | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
|
|
||
| - name: Notify on Release | ||
| if: steps.changesets.outputs.published == 'true' | ||
| run: | | ||
| echo "🎉 Published packages:" | ||
| echo "${{ steps.changesets.outputs.publishedPackages }}" | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -48,3 +48,4 @@ packages/blac/dist | |
|
|
||
| apps/docs/.vitepress/cache | ||
| apps/docs/.vitepress/dist | ||
| .serena | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| { | ||
| "packages": { | ||
| "include": [ | ||
| "packages/*", | ||
| "packages/plugins/**" | ||
| ], | ||
| "exclude": [ | ||
| "packages/test-utils", | ||
| "packages/internal-*" | ||
| ], | ||
| "pattern": "@blac/*" | ||
| }, | ||
| "publish": { | ||
| "defaultTag": "preview", | ||
| "access": "public", | ||
| "registry": "https://registry.npmjs.org/", | ||
| "gitChecks": false | ||
| }, | ||
| "build": { | ||
| "command": "pnpm run build", | ||
| "required": true | ||
| }, | ||
| "version": { | ||
| "gitTag": false, | ||
| "allowedBumps": ["patch", "minor", "major", "prerelease"], | ||
| "syncWorkspaceDependencies": true | ||
| }, | ||
| "hooks": { | ||
| "prepublish": "", | ||
| "postpublish": "" | ||
| } | ||
| } |
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.
steps.version.outputs.versionis undefined – manual release will failLines 66-69 reference an output called
version, but no step withid: versionexists. Action-lint rightfully flags this.One minimal fix is to capture the version after
changeset publish:Then reference
${{ steps.publish.outputs.version }}in the Create GitHub Release step.📝 Committable suggestion
🧰 Tools
🪛 YAMLlint (1.37.1)
[error] 60-60: trailing spaces
(trailing-spaces)
🤖 Prompt for AI Agents