Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Adjusting github workflows for CodeQL testing #520

Merged
merged 12 commits into from
May 6, 2024
Merged

Adjusting github workflows for CodeQL testing #520

merged 12 commits into from
May 6, 2024

Conversation

RafaelGCPP
Copy link
Contributor

@RafaelGCPP RafaelGCPP commented Mar 6, 2024 •
edited
Loading

  • Updated CodeQL action versions to v3
  • Updated minimum CMake requirement to 3.5
  • Split CodeQL analysis in two files, one for C++ and one for Python, as the "run" step was not being effective in the Python testing

With those modifications, now the code is tested and validated.

@RafaelGCPP RafaelGCPP mentioned this pull request Mar 7, 2024
Open
@RafaelGCPP
Copy link
Contributor Author

Related to Issue #521

@wb2osz
Copy link
Owner

wb2osz commented Mar 9, 2024

Thank you for these updates.
I've merged them into the dev branch and am awaiting the results.

73,
John WB2OSZ

@wb2osz wb2osz merged commit 69407cc into wb2osz:master May 6, 2024
6 checks passed
@wb2osz
Copy link
Owner

wb2osz commented May 6, 2024

It turns out this needs to be on master branch.
Merged to master branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@RafaelGCPP @wb2osz