jamulus icon indicating copy to clipboard operation
jamulus copied to clipboard

Resolve deprecation notice

Open pljones opened this issue 1 month ago • 2 comments

Short description of changes

Current builds warn:

Build for Linux .deb amd64 (artifacts+codeQL)
CodeQL Action v3 will be deprecated in December 2026. Please update all occurrences of the CodeQL Action in your workflow files to v4. For more information, see https://github.blog/changelog/2025-10-28-upcoming-deprecation-of-codeql-action-v3/

with a description on the referenced page of how to update. This change applies the description

CHANGELOG: SKIP

Context: Fixes an issue?

Deprecation notice

Does this change need documentation? What needs to be documented and how?

No.

Status of this Pull Request / What is missing until this pull request can be merged?

Let's see if the notice goes away...

Checklist

  • [x] I've verified that this Pull Request follows the general code principles
  • [ ] I tested my code and it does what I want
  • [ ] My code follows the style guide
  • [ ] I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • [ ] I've filled all the content above

AUTOBUILD: Please build all targets

pljones avatar Nov 23 '25 16:11 pljones

There's a warning in the CI. Not sure if it's relevant. Probably build with all targets here gave this error. --> try without in another PR

Warning: Code scanning cannot determine the alerts introduced by this pull request, because 1 configuration present on refs/heads/main was not found: Actions workflow (autobuild.yml)

❓  .github/workflows/autobuild.yml:release_assets/build_all_targets:false/config:

ann0see avatar Nov 28 '25 17:11 ann0see

There's a warning in the CI. Not sure if it's relevant. Probably build with all targets here gave this error. --> try without in another PR

I think it needs a CodeQL run against the target branch -- master -- for both configurations. So I guess we need this merged and run at least twice on master before we can check any other branches and not get warnings. (One other merge, with build_all_targets:false after this one, that is.)

pljones avatar Nov 30 '25 10:11 pljones