Conversation
This comment has been minimized.
This comment has been minimized.
Test Results131 tests ±0 131 ✅ ±0 31s ⏱️ -4s Results for commit 33fcd65. ± Comparison against base commit d276991. This pull request removes 9 and adds 9 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
/license-check |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
.github/workflows/quality.yml
Outdated
| - name: Eslint | ||
| run: cd web/textviewer && npm run lint | ||
|
|
||
| test-siteplan: |
There was a problem hiding this comment.
@mariusheine i think, we should make own workflow for all testing, also swtbot and this
There was a problem hiding this comment.
@TruongQuangSB Yeah good idea. I will adjust the workflows
This comment has been minimized.
This comment has been minimized.
License summary❌ Not yet vetted dependencies:
Workflow run (with attached summary files): |
| workflows: ["Build SET"] | ||
| types: [completed] | ||
| branches: | ||
| - '*' |
There was a problem hiding this comment.
@mariusheine Did you already test this Workflow on your fork ? I think this workflow should be triggered by "Build SET" workflow. When trigger by "Build SET" completed, then it can't take correct the artifact for test.
Otherweise should this Workflow only execute by main, normal PR and update package.json in siteplan
There was a problem hiding this comment.
@TruongQuangSB No I have not tested it by now. I wanted to trigger it only on completed because otherwise the SET build was not successful.
No description provided.