-
Notifications
You must be signed in to change notification settings - Fork 191
Experimental - v3 #1426
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?
Experimental - v3 #1426
Changes from all commits
237b8f6
77a74ae
030aebd
8daf57f
b57ecad
8a41b32
3d23352
451ab31
5e1f9db
8a88af3
77f2cb8
de4de19
1535bf7
f5a6742
551fc6f
80bfc19
2ca06ed
ed3738c
83a49f9
0653166
1823f31
99d9d57
c5ef988
e37b998
aff493c
f9b7f4c
52768ba
ebc14f2
9bb5af1
5bafb43
796f4ca
61892d2
72fbac8
cb72ada
0ab8b8c
6ed9295
9d48e24
02cad8b
6c7c491
683f905
1ab5740
f6cb96a
bf531a2
6b7fa24
2baccc1
c3d642f
f1f076b
9be6b37
f7cb602
71be4f5
be82609
c300993
3138166
be59d35
a618b06
519e7d9
40b5ee6
b649e10
ec74418
1d668ab
84219f7
928c390
9f4b125
9e25509
58dfc11
d79ec2d
1157297
90fee11
1efdf46
b6cd226
a7550bb
6ffc695
4b8aae9
2cf4909
ee7397c
6a3fa85
d44e4ea
0444b2c
00864a2
5b72a4a
89dae6c
148bd8d
e1390f7
1676563
32a5676
0985c2d
1c9711e
7539334
f403ad8
983db5b
b4ab82e
ff0b164
595a80d
aa38caf
9a23132
3c9d89a
e77c288
632f5cc
df757a1
e5f2bea
3c2792a
4dea087
7f0d6aa
a8656a4
e98658d
0f6adca
27d3c4b
43df72e
5c029b7
c8c4e9b
33b34da
dab47d7
443a867
4e33da2
917f712
33b092d
dc4068d
b61036f
300a4d3
370449a
00994fd
bbd75b4
0dfe477
538ac22
c8b6126
76625ba
a36ab22
cacf309
281f871
04a4a43
1ede071
8ae5620
c07584a
cfb4c63
f35cc81
d83885c
a73fe3d
1f91b56
699045c
c967cd2
665cc7f
f0558b3
fe3eb54
18820a0
5f25fed
6fde5be
990d6da
8ee6eb0
1c0bb49
b0ca196
9a75d91
4acb7c5
99779c3
6106361
ea1d104
5050b92
55e46fe
e5511fb
148282b
fff63bb
b600f8c
3ad0844
850dad1
3ea6d30
9628d62
f3859ed
4d5549d
5a4a199
78d5f67
1d20556
abc5fba
481a147
a6f81d1
2be2851
099e831
7a236c0
c3ceb8c
9ee3418
d0a4b19
98c050c
618ee9a
cf4995f
a282b52
a18c789
fc54b5e
9f3d1c1
c6f61c2
aac44fb
dc144fc
564fc17
ee8204a
af79280
b3ea9ab
3a8fd38
81c345b
4f3f607
e5304b5
4463627
ad2d110
a55d452
9004b05
640d5db
9e57c7b
213b2b4
fb2ff0f
3a1258a
e3851a7
37d13a1
2bfd5f4
3bf65b2
c4f62ea
4500e32
e274564
4673a21
3e8a066
e2bb5c7
69c1105
88bc400
255be1e
b444f95
37f79ed
56b0b97
5a3bc2c
f745cf4
d491773
e68fd78
36c2115
1164b9c
14c8093
6331fd7
f01345a
40f22ce
0d5401f
b0d09a0
86a15de
684658c
c3f0c83
20f2bbc
e511f42
aeb29f7
b45510f
dcad25c
7736a9c
ba6085d
7353f59
f9f4e0c
297c863
ab9db9c
4a718a3
cfee72e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,118 @@ | ||||||||||||||||||||||||||||||||
| name: running unit tests on pull request (v2) | ||||||||||||||||||||||||||||||||
| run-name: workflow ${{github.run_number}} of branch ${{github.head_ref}} | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| on: | ||||||||||||||||||||||||||||||||
| pull_request: | ||||||||||||||||||||||||||||||||
| branches: | ||||||||||||||||||||||||||||||||
| - main | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| jobs: | ||||||||||||||||||||||||||||||||
| Frontend-Tests: | ||||||||||||||||||||||||||||||||
| name: Frontend Tests | ||||||||||||||||||||||||||||||||
| runs-on: ubuntu-22.04 | ||||||||||||||||||||||||||||||||
| timeout-minutes: 20 | ||||||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||||||
| - name: 🔎 Check out repository code ... | ||||||||||||||||||||||||||||||||
| uses: actions/checkout@v4 | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: ⏳ Setup pnpm ... | ||||||||||||||||||||||||||||||||
| uses: pnpm/action-setup@v4 | ||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||
| version: 9.12.0 | ||||||||||||||||||||||||||||||||
| standalone: true | ||||||||||||||||||||||||||||||||
| run_install: false | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: ⏳ Setting Up Node ... | ||||||||||||||||||||||||||||||||
| uses: actions/setup-node@v4 | ||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||
| node-version: 20 | ||||||||||||||||||||||||||||||||
| cache: pnpm | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: ⏳ Install workspace dependencies ... | ||||||||||||||||||||||||||||||||
| run: pnpm install --frozen-lockfile | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🖥 typecheck Widget ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot/widget... run typecheck | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🖥 typecheck Frontend ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot/frontend... run typecheck | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🖋️ ESLint checks for widget ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot/widget... run lint | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🖋️ ESLint checks for Frontend ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot/frontend... run lint | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🛠️ build Widget ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot/widget... run build | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🛠️ build Frontend ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot/frontend... run build | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| CLI-Checks: | ||||||||||||||||||||||||||||||||
| name: CLI Checks | ||||||||||||||||||||||||||||||||
| runs-on: ubuntu-22.04 | ||||||||||||||||||||||||||||||||
| timeout-minutes: 20 | ||||||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||||||
| - name: 🔎 Check out repository code ... | ||||||||||||||||||||||||||||||||
| uses: actions/checkout@v4 | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: ⏳ Setup pnpm ... | ||||||||||||||||||||||||||||||||
| uses: pnpm/action-setup@v4 | ||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||
| version: 9.12.0 | ||||||||||||||||||||||||||||||||
| standalone: true | ||||||||||||||||||||||||||||||||
| run_install: false | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: ⏳ Setting Up Node ... | ||||||||||||||||||||||||||||||||
| uses: actions/setup-node@v4 | ||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||
| node-version: 20 | ||||||||||||||||||||||||||||||||
| cache: pnpm | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: ⏳ Install workspace dependencies ... | ||||||||||||||||||||||||||||||||
| run: pnpm install --frozen-lockfile | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🖥 typecheck CLI ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot-ai/cli... run typecheck | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🖋️ ESLint checks for CLI ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot-ai/cli... run lint | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🔬 running unit tests for CLI ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot-ai/cli... run test | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🛠️ build CLI ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot-ai/cli... run build | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| API-Tests: | ||||||||||||||||||||||||||||||||
|
Comment on lines
+53
to
+88
Check warningCode scanning / CodeQL Workflow does not contain permissions Medium
Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}
Copilot AutofixAI 4 days ago To fix the problem, add an explicit permissions:
contents: readNo further changes, imports, or definitions are required for this edit.
Suggested changeset
1
.github/workflows/v2-pull_request.yml
Copilot is powered by AI and may make mistakes. Always verify output.
Positive FeedbackNegative Feedback
Refresh and try again.
|
||||||||||||||||||||||||||||||||
| runs-on: ubuntu-22.04 | ||||||||||||||||||||||||||||||||
| timeout-minutes: 20 | ||||||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||||||
| - name: 🔎 Check out repository code ... | ||||||||||||||||||||||||||||||||
| uses: actions/checkout@v4 | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: ⏳ Setup pnpm ... | ||||||||||||||||||||||||||||||||
| uses: pnpm/action-setup@v4 | ||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||
| version: 9.12.0 | ||||||||||||||||||||||||||||||||
| standalone: true | ||||||||||||||||||||||||||||||||
| run_install: false | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: ⏳ Setting Up Node ... | ||||||||||||||||||||||||||||||||
| uses: actions/setup-node@v4 | ||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||
| node-version: 20 | ||||||||||||||||||||||||||||||||
| cache: pnpm | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: ⏳ Install workspace dependencies ... | ||||||||||||||||||||||||||||||||
| run: pnpm install --frozen-lockfile | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🖥 typecheck API ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot/api... run typecheck | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🖋️ ESLint checks for API ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot/api... run lint | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| - name: 🔬 running unit tests for API ... | ||||||||||||||||||||||||||||||||
| run: pnpm --filter @hexabot/api... run test | ||||||||||||||||||||||||||||||||
|
Comment on lines
+89
to
+118
Check warningCode scanning / CodeQL Workflow does not contain permissions Medium
Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}
Copilot AutofixAI 4 days ago To fix this issue, you should add an explicit permissions:
contents: readNo additional imports or definitions are necessary. If, in the future, some job needs additional access (e.g.,
Suggested changeset
1
.github/workflows/v2-pull_request.yml
Copilot is powered by AI and may make mistakes. Always verify output.
Positive FeedbackNegative Feedback
Refresh and try again.
|
||||||||||||||||||||||||||||||||
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium
Copilot Autofix
AI 4 days ago
The problem is that the workflow does not specify a
permissionsblock, so jobs may run with more permissions than required. To fix this, add apermissionskey at the root of the workflow YAML file, just after therun-nameand before theon:block. For this workflow, all jobs only read from the repository for testing and do not need any write permissions. Set all permissions to minimum required, which iscontents: read. If you later add jobs that need to write pull request comments, you can expand permissions per-job or at the global level. Edit the.github/workflows/v2-pull_request.ymlfile to insert the following block:right after the
run-nameblock (line 2).