ci: Fix static analysis errors #955
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lfrancke
approved these changes
Dec 13, 2024
NickLarsenNZ
pushed a commit
that referenced
this pull request
Dec 13, 2024
* ci: Fix high severity errors * ci: Set persist-credentials to false * ci: Move step output to env var * chore: Add changelog entry
NickLarsenNZ
added a commit
that referenced
this pull request
Dec 13, 2024
* ci: remove redundant usages of the free-disk-space action Co-authored-by: Techassi <[email protected]> * ci: Add attempt number to the workflow run name, and URL in the slack message (#941) Co-authored-by: Techassi <[email protected]> * ci: register a dummy test_build_images workflow which will be completed in a separate PR (#947) * Use our build-repo for Maven downloads & other updates (#953) * Use our build-repo for Maven downloads * ci(build): Continue matrix builds on failure so there is less jobs to rerun when there are transient failures * Update Spark Mvn build as well * Hopefully silence Spark build * Update FMPP version This is because FMPP Maven Plugin depends on FMPP in version 0.9.14 which itself depends on a Freemarker version that has not been pinned. Instead it specifies a "range" which resolves to a SNAPSHOT version which we don't want. * Add Stackable repo to Kafka build as well * Update changelog * Update CHANGELOG.md Co-authored-by: Nick <[email protected]> --------- Co-authored-by: Nick Larsen <[email protected]> Co-authored-by: Nick <[email protected]> * ci: Fix static analysis errors (#955) * ci: Fix high severity errors * ci: Set persist-credentials to false * ci: Move step output to env var * chore: Add changelog entry * ci(build): stagger the scheduled builds to ease load on external systems (#954) --------- Co-authored-by: Techassi <[email protected]> Co-authored-by: Lars Francke <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The workflows were checked with https://github.com/woodruffw/zizmor to find various security related improvements.
The following lints were fixed: