-
Notifications
You must be signed in to change notification settings - Fork 2
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
Resolve issues on workflows #1038
base: main
Are you sure you want to change the base?
Conversation
Merging to
|
Dependency ReviewThe following issues were found:
License Issues.github/workflows/cmake-requirements.txt
pages/package.json
OpenSSF ScorecardScorecard details
Scanned Manifest Files.github/workflows/ci.yml
.github/workflows/cmake-requirements.txt
.github/workflows/conan-requirements.txt
.github/workflows/styles.yml
pages/package.json
.github/workflows/pages.yml
package-lock.json
|
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.
Scorecard found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
🦙 MegaLinter status: ❌ ERROR
See detailed report in MegaLinter reports You could have same capabilities but better runtime performances if you request a new MegaLinter flavor. |
name: 💎 CLang-Format | ||
runs-on: ubuntu-latest | ||
permissions: | ||
contents: write |
Check failure
Code scanning / Scorecard
Token-Permissions High
Remediation tip: Visit https://app.stepsecurity.io/secureworkflow.
Tick the 'Restrict permissions for GITHUB_TOKEN'
Untick other options
NOTE: If you want to resolve multiple issues at once, you can visit https://app.stepsecurity.io/securerepo instead.
Click Remediation section below for further remediation help
name: 💎 CMake-Format | ||
runs-on: ubuntu-latest | ||
permissions: | ||
contents: write |
Check failure
Code scanning / Scorecard
Token-Permissions High
Remediation tip: Visit https://app.stepsecurity.io/secureworkflow.
Tick the 'Restrict permissions for GITHUB_TOKEN'
Untick other options
NOTE: If you want to resolve multiple issues at once, you can visit https://app.stepsecurity.io/securerepo instead.
Click Remediation section below for further remediation help
permissions: | ||
actions: read | ||
contents: read | ||
security-events: write |
Check failure
Code scanning / Scorecard
Token-Permissions High
Remediation tip: Visit https://app.stepsecurity.io/secureworkflow.
Tick the 'Restrict permissions for GITHUB_TOKEN'
Untick other options
NOTE: If you want to resolve multiple issues at once, you can visit https://app.stepsecurity.io/securerepo instead.
Click Remediation section below for further remediation help
cinst ninja ccache | ||
- name: 🧊 ⚙️ Configure MSVC | ||
if: runner.os == 'Windows' | ||
uses: ilammy/msvc-dev-cmd@ 0b201ec74fa43914dc39ae48a89fd1d8cb592756 # v1.13.0 |
Check warning
Code scanning / Scorecard
Pinned-Dependencies Medium
Click Remediation section below to solve this issue
conan profile show default | ||
- name: (Linux/macOS) restore ccache | ||
if: runner.os == 'Linux' || runner.os == 'macOS' | ||
uses: pat-s/[email protected] |
Check warning
Code scanning / Scorecard
Pinned-Dependencies Medium
Click Remediation section below to solve this issue
"$GITHUB_WORKSPACE/appimaged-*.AppImage" "${APPIMAGE_DST_PATH}" | ||
- name: 🐧 📤 Upload Linux artifacts | ||
if: runner.os == 'Linux' | ||
uses: actions/upload-artifact@v4 |
Check warning
Code scanning / Scorecard
Pinned-Dependencies Medium
Click Remediation section below to solve this issue
permissions: | ||
actions: read | ||
contents: read | ||
security-events: write |
Check failure
Code scanning / Scorecard
Token-Permissions High
Remediation tip: Visit https://app.stepsecurity.io/secureworkflow.
Tick the 'Restrict permissions for GITHUB_TOKEN'
Untick other options
NOTE: If you want to resolve multiple issues at once, you can visit https://app.stepsecurity.io/securerepo instead.
Click Remediation section below for further remediation help
-DBUILD_SHARED_LIBS:BOOL="TRUE" \ | ||
-DENABLE_CACHE="ON" | ||
- name: 📦 🍎 Build and package application (macdeployqt and zipfile) | ||
# TODO: Stored in install. Build DMG with CPack |
Check notice
Code scanning / devskim
A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note
cmake --build . --config ${{matrix.build_type}} | ||
cpack -C "${{matrix.build_type}}" -G "DragNDrop;ZIP" | ||
- name: ⚙️ 🧊 Build | ||
# TODO: Stored in install. Build installer with CPack |
Check notice
Code scanning / devskim
A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note
env: | ||
APPIMAGETOOL_ARCH: x86_64 | ||
- name: 🐧 📦 Create AppImage | ||
# TODO: Stored in build directory |
Check notice
Code scanning / devskim
A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note
@@ -1,4 +1,4 @@ | |||
--- | |||
# TODO: Update to use Qt6.x |
Check notice
Code scanning / devskim
A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note
… into workflow-issues
… into workflow-issues
|
||
\brief Destructor for DataContext() | ||
\details Close the connection to the database. | ||
// TODO: Add Constructor: DataContext(const QString &name, + init for mySQL |
Check notice
Code scanning / devskim
A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note
\details Close the connection to the database. | ||
// TODO: Add Constructor: DataContext(const QString &name, + init for mySQL | ||
// etc.) | ||
// TODO: Remove access of settings |
Check notice
Code scanning / devskim
A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note
cache: pip | ||
- name: 🟨 Install cmakelang | ||
run: | | ||
python -m pip install --upgrade pip |
Check warning
Code scanning / Scorecard
Pinned-Dependencies Medium
Click Remediation section below to solve this issue
- name: 🟨 Install cmakelang | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip3 install -r .github/workflows/cmake-requirements.txt |
Check warning
Code scanning / Scorecard
Pinned-Dependencies Medium
Click Remediation section below to solve this issue
@@ -0,0 +1,3 @@ | |||
<?xml version="1.0" encoding="utf-8"?> |
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.
Parsing error: Unexpected token <
@@ -0,0 +1,3 @@ | |||
<?xml version="1.0" encoding="utf-8"?> | |||
<!DOCTYPE TS> | |||
<TS/> |
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.
Missing semicolon
Type assertion using the '<>' syntax is forbidden. Use the 'as' syntax instead.
missing whitespace
@@ -0,0 +1,3 @@ | |||
<?xml version="1.0" encoding="utf-8"?> | |||
<!DOCTYPE TS> |
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.
Type assertion using the '<>' syntax is forbidden. Use the 'as' syntax instead.
missing whitespace
@@ -0,0 +1,3 @@ | |||
<?xml version="1.0" encoding="utf-8"?> |
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.
Missing semicolon
Type assertion using the '<>' syntax is forbidden. Use the 'as' syntax instead.
missing whitespace
unused expression, expected an assignment or function call
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.
check-spelling found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
@check-spelling-bot Report🔴 Please reviewSee the 📂 files view, the 📜action log, or 📝 job summary for details. Unrecognized words (2141)
Some files were automatically ignored 🙈These sample patterns would exclude them:
You should consider excluding directory paths (e.g. You should consider adding them to:
File matching is via Perl regular expressions. To check these files, more of their words need to be in the dictionary than not. You can use To accept these unrecognized words as correct and update file exclusions, you could run the following commands... in a clone of the [email protected]:jmuelbert/jmbde-QT.git repository curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/00c989c97749eb0cb2d256bdc55ac61b0096c6d3/apply.pl' |
perl - 'https://github.com/jmuelbert/jmbde-QT/actions/runs/11651012004/attempts/1' OR To have the bot accept them for you, reply quoting the following line: Available 📚 dictionaries could cover words not in the 📘 dictionary
Consider adding them (in with:
extra_dictionaries:
cspell:python/src/python/python-lib.txt
cspell:python/src/python/python.txt
cspell:python/src/common/extra.txt
cspell:cpp/src/ecosystem.txt
cspell:php/dict/php.txt To stop checking additional dictionaries, add (in check_extra_dictionaries: '' Pattern suggestions ✂️ (37)You could add these patterns to
Errors (7)See the 📂 files view, the 📜action log, or 📝 job summary for details.
See ❌ Event descriptions for more information. If the flagged items are 🤯 false positivesIf items relate to a ...
🚂 If you're seeing this message and your PR is from a branch that doesn't have |
Description
Many workflows doesn't run.
Related Issues / Pull Requests
#1006 #1007 #1008 #1009 #1010 #1011 #1012
Motivation and Context
Make the code better and less the errors
Additional text