-
Notifications
You must be signed in to change notification settings - Fork 48
Next #325
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: main
Are you sure you want to change the base?
Next #325
Changes from all commits
558e460
85acd09
c165ec2
e10aaa8
45189ad
247af49
7a52c75
6325730
6f0c94b
9081fa8
535121c
db071ac
4f3c872
fd496b1
09b83f7
b0110ff
b5c8591
34671e4
18909bc
7e90824
65f97c8
6bf3dd0
9f28c76
09eb531
0478e12
0c4d6e1
185ee47
44385d1
36dc3f2
59e81f6
0b77850
0bad73d
2a06487
0f02493
f2c492e
c993493
23b0b61
4d9e4bf
60a0d5c
dffcf86
98d8dd0
0f90bf1
2cf27e0
79d8d17
7a84c31
18188b7
2a08cf6
9659e5f
52e6f45
37da8fb
16f0beb
a7a9b5d
40fb564
6ef18ae
101733c
d735c9a
abc3783
d620281
a02594a
196896c
780505d
2ae5c26
6719952
7da6b86
062d3ec
86cdb73
b7e19d8
fb8b1ed
8a3316e
1518a0f
83942c1
c088e45
3d9d31f
1955660
30fa057
3b50c35
ce2b249
1489b16
317dd3f
40105cc
efde1ec
136de71
7e2f4da
d8326d0
aceaca7
cdb58ac
e1c4a38
034243e
7ddce03
0d30abc
0fefd92
dd980f7
2ed1250
218be58
952b37e
fafc942
aca40bd
6fd5ef1
65e03a4
4e4e31d
ba96092
28b9e27
263a9e2
2637bd0
737010f
2c4b5cf
c4033d9
00be49b
5b11372
f4da718
b6d23a2
6be70dc
66e8127
9552823
4a33959
2037d5e
c6caab9
f2198f1
383f5e6
e3b67a2
3a06f4c
7ccbbff
4e5cc18
dcfb9e1
077179c
0a65051
8f82877
dc65e5f
3a17fe8
72e5104
d1684a1
754a996
411470c
0bdb5e2
8a68cf8
4d6bdef
eab33ba
a90a620
8c93290
d030e8e
1202c02
226ed26
f95ebba
8a9c15a
893bed8
c9aac79
f06f46a
f65ab98
6faeea0
0fb04d1
a3c6dfe
b429c6a
3e84c1c
524d1cd
189c216
3fa55f7
10305c4
c342c98
ad1222b
621bb1e
d4b1eff
a301053
44da052
325fef8
e3004a6
1b21f45
e2a6d86
71bdaf8
d2c20e8
50b60cc
e0bb695
2acd23f
bd11254
d339bb2
d3b1b22
ad30eb1
634a501
5da0018
d4ec91b
1aace7d
5a2588f
b59aa24
79d81a8
144340d
a447787
2b4bc90
d8a3234
46f65c7
293b6a8
f802054
6ca319c
fc3a419
85d1b43
a5b3436
53208cd
c67ae13
976f4b6
460d19b
c2c15c4
b9bf637
f61dd8d
b92d102
d40a62b
a438881
7c771ee
acc5cbc
062531d
de873fd
08506a2
7361c25
d94920e
cf1b6a2
41aedbb
abfd081
b1988af
c4d7bc0
1a491f0
f5527f5
a9f3d68
e54db8a
4472521
4d60711
df49e0d
a3ac011
e5b1d7e
ae9e7d5
73fb121
a8bd38d
bd82d9a
1bf564d
2421381
c255e7b
ddb9a4b
5b26bb8
da72a8b
77763d1
33845eb
6916470
2e88012
53df8e1
c2b4379
3837467
60d3605
8e32f56
dd70086
c15b4be
865e2fc
cb016bd
cfa2920
fe2477e
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,2 @@ | ||
| node_modules/ | ||
| !dist |
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,39 @@ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| name: Publish to pkg.vc | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| on: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| pull_request: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| permissions: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| pull-requests: write | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| contents: read | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| jobs: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| publish: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| runs-on: ubuntu-latest | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Checkout code | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: actions/checkout@v4 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Use Node.js | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: actions/setup-node@v2 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| node-version: 20 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+16
to
+19
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 Upgrade setup-node to v4 and enable pnpm cache. actionlint flags v2 as too old; caching speeds CI. - - name: Use Node.js
- uses: actions/setup-node@v2
- with:
- node-version: 20
+ - name: Use Node.js
+ uses: actions/setup-node@v4
+ with:
+ node-version: '20'
+ cache: 'pnpm'📝 Committable suggestion
Suggested change
🧰 Tools🪛 actionlint (1.7.7)17-17: the runner of "actions/setup-node@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue (action) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Install pnpm | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: pnpm/action-setup@v4 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Install dependencies | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| run: pnpm install --frozen-lockfile | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Build packages | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| run: pnpm run build | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Publish @appwrite.io/pink-svelte | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: pkg-vc/publish-action@main | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| organization: appwrite | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| directory: ./v2/pink-sb | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| secret: ${{ secrets.TRY_MODULE_SECRET }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Publish @appwrite.io/pink-icons-svelte | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: pkg-vc/publish-action@main | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| organization: appwrite | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| directory: ./v2/pink-icons | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| secret: ${{ secrets.TRY_MODULE_SECRET }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+26
to
+39
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 Guard publish steps or job against PR events. If you keep PR triggers for dry-runs, ensure secrets aren’t required. Otherwise gate the job. - publish:
+ publish:
+ if: github.event_name != 'pull_request'
runs-on: ubuntu-latest📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,24 +1,38 @@ | ||||||||||||||||||||||||||
| name: Tests | ||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| on: | ||||||||||||||||||||||||||
| push: | ||||||||||||||||||||||||||
| branches: [main] | ||||||||||||||||||||||||||
| pull_request: | ||||||||||||||||||||||||||
| branches: [main] | ||||||||||||||||||||||||||
| push: | ||||||||||||||||||||||||||
| branches: [main] | ||||||||||||||||||||||||||
| pull_request: | ||||||||||||||||||||||||||
| branches: ['**'] | ||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| jobs: | ||||||||||||||||||||||||||
| build: | ||||||||||||||||||||||||||
| runs-on: ubuntu-latest | ||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||
| - uses: actions/checkout@v2 | ||||||||||||||||||||||||||
| - name: Use Node.js | ||||||||||||||||||||||||||
| uses: actions/setup-node@v2 | ||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||
| node-version: 16 | ||||||||||||||||||||||||||
| - name: Install dependencies | ||||||||||||||||||||||||||
| run: npm ci | ||||||||||||||||||||||||||
| - name: Run Linter | ||||||||||||||||||||||||||
| run: npm run lint | ||||||||||||||||||||||||||
| continue-on-error: true | ||||||||||||||||||||||||||
| - name: Build packages | ||||||||||||||||||||||||||
| run: npm run build | ||||||||||||||||||||||||||
| test: | ||||||||||||||||||||||||||
| runs-on: ubuntu-latest | ||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||
| - uses: actions/checkout@v2 | ||||||||||||||||||||||||||
| - name: Use Node.js | ||||||||||||||||||||||||||
| uses: actions/setup-node@v2 | ||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||
| node-version: 20 | ||||||||||||||||||||||||||
|
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. Upgrade core actions to v4 (v2 is EOL on modern runners). checkout@v2 and setup-node@v2 will fail on current GitHub-hosted runners. Move to v4 and enable pnpm cache. - - uses: actions/checkout@v2
+ - uses: actions/checkout@v4
- name: Use Node.js
- uses: actions/setup-node@v2
+ uses: actions/setup-node@v4
with:
node-version: 20
+ cache: 'pnpm'
+ cache-dependency-path: pnpm-lock.yaml📝 Committable suggestion
Suggested change
🧰 Tools🪛 actionlint (1.7.7)13-13: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue (action) 15-15: the runner of "actions/setup-node@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue (action) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||
| - name: Install pnpm | ||||||||||||||||||||||||||
| uses: pnpm/action-setup@v4 | ||||||||||||||||||||||||||
| - name: Install dependencies | ||||||||||||||||||||||||||
| run: pnpm install --frozen-lockfile | ||||||||||||||||||||||||||
| - name: Build packages | ||||||||||||||||||||||||||
| run: pnpm run build | ||||||||||||||||||||||||||
| - name: Run Linter | ||||||||||||||||||||||||||
| run: pnpm run lint | ||||||||||||||||||||||||||
| - name: Run Static Analysis | ||||||||||||||||||||||||||
| run: pnpm run check | ||||||||||||||||||||||||||
| # - name: Run tests | ||||||||||||||||||||||||||
| # run: | | ||||||||||||||||||||||||||
| # pnpm dev & | ||||||||||||||||||||||||||
| # pnpm run test | ||||||||||||||||||||||||||
| # - uses: actions/upload-artifact@v4 | ||||||||||||||||||||||||||
| # if: ${{ !cancelled() }} | ||||||||||||||||||||||||||
| # with: | ||||||||||||||||||||||||||
| # name: genauigkeit-report | ||||||||||||||||||||||||||
| # path: | | ||||||||||||||||||||||||||
| # v2/pink-sb/.genauigkeit/report/index.html | ||||||||||||||||||||||||||
| # retention-days: 30 | ||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,4 +5,6 @@ icons-dist/ | |
| *.local | ||
| .turbo | ||
| .vercel | ||
| .idea | ||
| .idea | ||
| .zed | ||
| .vscode | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| engine-strict=true | ||
| auto-install-peers=true | ||
| legacy-peer-deps=true | ||
| node-linker=hoisted |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| { | ||
| "useTabs": false, | ||
| "tabWidth": 4, | ||
| "singleQuote": true, | ||
| "trailingComma": "none", | ||
| "printWidth": 100 | ||
| } |
This file was deleted.
This file was deleted.
This file was deleted.
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.
🛠️ Refactor suggestion
Do not publish on pull_request. Restrict to main/tags (prevents accidental releases).
Publishing on PRs risks unmerged or fork code attempting to publish. Trigger on push to main/tags and allow manual runs.
📝 Committable suggestion
🤖 Prompt for AI Agents