-
Notifications
You must be signed in to change notification settings - Fork 46
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
efb9310
a9d6b0e
3228c15
24b32b8
267da11
973fff2
b2e9b87
742f8b2
8df8c98
bc94ff8
813ff24
78fdff7
1a03ce8
5ba20ca
ad1b5c4
79a6929
44e5eb1
94df3da
3b35c69
c8081ce
f00b8c5
9e1c50a
ab3c221
e21c1d9
74cf26b
7254e0d
819eb7d
e1e883a
01ad18b
fee5c53
a75c2d5
f0ff26b
95ce490
543ebea
255fab5
373d6a6
a6874bc
2f1e67d
e91326e
f351106
ffd1c00
244d5db
6aed15d
edccb3a
e2eb987
3bfe2c7
7e2deae
d6705b9
0ed6a5f
6454219
7f8f433
0a9df07
d9ba720
3d145d4
2acaff9
b9e4bed
dbbb00f
95bdb79
ac94143
d7034e5
f36ef60
872efe2
dc53e79
c071257
8057bf6
c5b4d80
b12ef1f
7025bb6
9132446
7ac7261
19f15c8
c88c338
5ee0b63
e0df9cb
99ee9f5
96e10aa
8892b0f
7daf439
54f2e83
b5bb595
ebaa13e
3ea76a5
5913d0c
963d0b9
cc69c76
0a9015e
3d31cea
0cc4af0
14c6853
99f5ea4
a209351
fb7591b
260eb79
a7a43d6
95dc1c9
97e114e
142a846
18ed891
a64ab34
51326d1
f4bae29
bfa834d
1bc7cf7
da8cd99
3b4734f
a53ff9c
4ac7e2f
31a9048
fce7eac
9da8171
7ab8acd
25a3ab9
cf90d64
5f582f9
bc41d0c
d303221
23c182d
7f47803
24930a2
c880f34
e6b154b
3f81097
0468c93
753f9c0
950fa5e
0c51a94
b00fb9c
0c30e2f
363dcac
c0cdd91
b4e47f3
cad4411
da2ca90
206f46f
c1b4460
19eed0b
182bbaa
fd15f6e
828a552
d93a5e9
b023951
3f2602a
2a50f4b
8675026
c467db8
10eee24
698efda
0fed23b
28cba8a
8076873
fe36900
202a180
8672d4d
fcd953f
558e460
85acd09
c165ec2
e10aaa8
45189ad
247af49
7a52c75
6325730
6f0c94b
9081fa8
535121c
db071ac
4f3c872
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
d620281
196896c
2ae5c26
7da6b86
86cdb73
b7e19d8
8a3316e
1518a0f
83942c1
3d9d31f
1955660
30fa057
3b50c35
ce2b249
317dd3f
40105cc
efde1ec
136de71
7e2f4da
aceaca7
cdb58ac
034243e
7ddce03
0d30abc
dd980f7
2ed1250
218be58
2c4b5cf
c4033d9
b6d23a2
6be70dc
9552823
2037d5e
c6caab9
383f5e6
e3b67a2
3a06f4c
7ccbbff
dcfb9e1
0a65051
dc65e5f
3a17fe8
72e5104
d1684a1
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,36 @@ | ||||||||||||||||||||||||||
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 tests | ||||||||||||||||||||||||||
run: | | ||||||||||||||||||||||||||
pnpm dev & | ||||||||||||||||||||||||||
pnpm run test | ||||||||||||||||||||||||||
Comment on lines
+26
to
+29
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 Wait for dev server to be ready to avoid flaky tests. Start server, wait on a health URL, then run tests and clean up. - - name: Run tests
- run: |
- pnpm dev &
- pnpm run test
+ - name: Run tests
+ run: |
+ pnpm dev & SERVER_PID=$!
+ npx wait-on http://localhost:5173
+ pnpm run test
+ kill $SERVER_PID If the URL/port differs, replace it accordingly. 📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents
|
||||||||||||||||||||||||||
- 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