Add missing workflows #856

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

Open

bonzofenix wants to merge 12 commits into main from add-missing-workflows

Contributor

bonzofenix commented Nov 21, 2025

No description provided.

bonzofenix added 3 commits

November 21, 2025 16:45


          Add CodeQL analysis workflow for Go, Java, and Ruby on push, pull req…

ed73fab

…uest, and schedule


          Add Java CI lint workflow with code formatting and checkstyle validation

18a1dcb


          Add Dependency Updates Post-Processing GitHub Workflow

4f5ccc4

 • Introduce a new GitHub Actions workflow for post-processing dependency updates.
 • Trigger workflow on pull requests affecting specific paths and on manual dispatch.
 • Configure git with the author details of the last commit.
 • Utilize devbox-install-action for setting up the development environment.
 • Implement steps to tidy Go modules and update package specifications.
 • Ensure no redundant workflow runs by checking the last commit message.
 • Handle submodule deinitialization and cleanup for branch switching.
 • Add conditional logic to commit and push changes if necessary files are modified.

Contributor

github-advanced-security bot commented Nov 21, 2025

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.


           Update dependency update workflow and Makefile for UAAC Nix package

2a4bea4

 - Modify dependency update workflow triggers to include root and all subdirectory go.mod and go.sum files
 - Add `update-uaac-nix-package` target to Makefile for updating the UAAC Nix package

bonzofenix self-assigned this

bonzofenix added the enhancement label

bonzofenix and others added 5 commits

November 24, 2025 11:04


           Update CodeQL languages and add Go checkstyle formatter

d45600d

 - Modify CodeQL analysis workflow to only include 'go' and 'java' languages, removing 'ruby'.
 - Add a new Go program for formatting checkstyle results, capable of parsing XML and outputti
 formatted error messages.


          Updated workflow to simplify go mod tidy handling

449651e

p
removes src/autoscaler submodule references ported from
app-autoscaler-release workflow


          Add go-mod-tidy to dependency post-processing workflow

bd91b64


          🤖🦾🛠️ go mod tidy

cea5e32


          Adds missing styleguide folder

bonzofenix force-pushed the add-missing-workflows branch from eacc8ea to 7616172 Compare

November 24, 2025 12:09

github-actions bot reviewed

View reviewed changes

style-guide/inspect-java-format-0.1.sh

+              #   Checkstyle                                                      #
+              #####################################################################
+              echo "Running Checkstyle using $DOWNLOAD_PATH"/"$CHECKSTYLE_JAR_NAME..."
+              trap "rm ${result_log} || echo " EXIT

Contributor

github-actions bot Nov 24, 2025

⚠️ [shellcheck] _{reported by reviewdog 🐶}
Use single quotes, otherwise this expands now rather than when signalled. SC2064

style-guide/inspect-java-format-0.1.sh

+              #####################################################################
+              echo "Running Checkstyle using $DOWNLOAD_PATH"/"$CHECKSTYLE_JAR_NAME..."
+              trap "rm ${result_log} || echo " EXIT
+              CHECKSTYLE_OUTPUT=$(java \

Contributor

github-actions bot Nov 24, 2025

⚠️ [shellcheck] _{reported by reviewdog 🐶}
CHECKSTYLE_OUTPUT appears unused. Verify use (or export if used externally). SC2034

style-guide/inspect-java-format-0.1.sh

+                -jar "$DOWNLOAD_PATH"/"$CHECKSTYLE_JAR_NAME" \
+                -p "${CHECKSTYLE_CONFIG_PATH}"/checkstyle.properties \
+                -c "${CHECKSTYLE_CONFIG_PATH}"/"${CHECKSTYLE_CONFIG_FILE_NAME}" \
+                -o "${result_log}" $changed_java_files)

Contributor

github-actions bot Nov 24, 2025

📝 [shellcheck] _{reported by reviewdog 🐶}
Double quote to prevent globbing and word splitting. SC2086

style-guide/inspect-java-format-0.1.sh

+                                  --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED \
+                                  --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED \
+                                  --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED \
+                                  -jar "$DOWNLOAD_PATH"/"$GOOGLE_JAR_NAME"  -n --skip-javadoc-formatting $changed_java_files)

Contributor

github-actions bot Nov 24, 2025

📝 [shellcheck] _{reported by reviewdog 🐶}
Double quote to prevent globbing and word splitting. SC2086

style-guide/inspect-java-format-0.1.sh

+                  echo -e "These file(s) have formatting issues: \n$files_to_be_formatted\n"
+                  files_to_be_formatted=$(echo "$files_to_be_formatted" |tr '\n' ' ')
+                  echo "Please correct the formatting of the files(s) using one of the following options:"
+                  echo "   java --add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED  -jar "$DOWNLOAD_PATH"/"$GOOGLE_JAR_NAME" -replace --skip-javadoc-formatting $files_to_be_formatted"

Contributor

github-actions bot Nov 24, 2025

⚠️ [shellcheck] _{reported by reviewdog 🐶}
The surrounding quotes actually unquote this. Remove or escape them. SC2027

style-guide/inspect-java-format-0.1.sh

+                  echo -e "These file(s) have formatting issues: \n$files_to_be_formatted\n"
+                  files_to_be_formatted=$(echo "$files_to_be_formatted" |tr '\n' ' ')
+                  echo "Please correct the formatting of the files(s) using one of the following options:"
+                  echo "   java --add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED  -jar "$DOWNLOAD_PATH"/"$GOOGLE_JAR_NAME" -replace --skip-javadoc-formatting $files_to_be_formatted"

Contributor

github-actions bot Nov 24, 2025

📝 [shellcheck] _{reported by reviewdog 🐶}
Double quote to prevent globbing and word splitting. SC2086

style-guide/inspect-java-format-0.1.sh

+                  echo -e "These file(s) have formatting issues: \n$files_to_be_formatted\n"
+                  files_to_be_formatted=$(echo "$files_to_be_formatted" |tr '\n' ' ')
+                  echo "Please correct the formatting of the files(s) using one of the following options:"
+                  echo "   java --add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED  -jar "$DOWNLOAD_PATH"/"$GOOGLE_JAR_NAME" -replace --skip-javadoc-formatting $files_to_be_formatted"

Contributor

github-actions bot Nov 24, 2025

⚠️ [shellcheck] _{reported by reviewdog 🐶}
The surrounding quotes actually unquote this. Remove or escape them. SC2027

github-actions bot reviewed

View reviewed changes

style-guide/inspect-java-format-0.1.sh

+                -jar "$DOWNLOAD_PATH"/"$CHECKSTYLE_JAR_NAME" \
+                -p "${CHECKSTYLE_CONFIG_PATH}"/checkstyle.properties \
+                -c "${CHECKSTYLE_CONFIG_PATH}"/"${CHECKSTYLE_CONFIG_FILE_NAME}" \
+                -o "${result_log}" $changed_java_files)

Contributor

github-actions bot Nov 24, 2025

[shellcheck (suggestion)] _{reported by reviewdog 🐶}

Suggested change

      
              -o "${result_log}" $changed_java_files)
          
              -o "${result_log}" "$changed_java_files")

style-guide/inspect-java-format-0.1.sh

+                                  --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED \
+                                  --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED \
+                                  --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED \
+                                  -jar "$DOWNLOAD_PATH"/"$GOOGLE_JAR_NAME"  -n --skip-javadoc-formatting $changed_java_files)

Contributor

github-actions bot Nov 24, 2025

[shellcheck (suggestion)] _{reported by reviewdog 🐶}

Suggested change

      
                                -jar "$DOWNLOAD_PATH"/"$GOOGLE_JAR_NAME"  -n --skip-javadoc-formatting $changed_java_files)
          
                                -jar "$DOWNLOAD_PATH"/"$GOOGLE_JAR_NAME"  -n --skip-javadoc-formatting "$changed_java_files")

style-guide/inspect-java-format-0.1.sh

+                  echo -e "These file(s) have formatting issues: \n$files_to_be_formatted\n"
+                  files_to_be_formatted=$(echo "$files_to_be_formatted" |tr '\n' ' ')
+                  echo "Please correct the formatting of the files(s) using one of the following options:"
+                  echo "   java --add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED  -jar "$DOWNLOAD_PATH"/"$GOOGLE_JAR_NAME" -replace --skip-javadoc-formatting $files_to_be_formatted"

Contributor

github-actions bot Nov 24, 2025

[shellcheck (suggestion)] _{reported by reviewdog 🐶}

Suggested change

      
                echo "   java --add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED  -jar "$DOWNLOAD_PATH"/"$GOOGLE_JAR_NAME" -replace --skip-javadoc-formatting $files_to_be_formatted"
          
                echo "   java --add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED  -jar ""$DOWNLOAD_PATH""/"$GOOGLE_JAR_NAME" -replace --skip-javadoc-formatting $files_to_be_formatted"

github-actions bot reviewed

View reviewed changes

style-guide/README.md

		@@ -0,0 +1,142 @@
		## Code Quality
		Autoscaler uses [Pre-commit](https://pre-commit.com/) for keeping our code base clean. It is a package manager fo git hooks.

Contributor

github-actions bot Nov 24, 2025

⚠️ [alex] _{reported by reviewdog 🐶}
Be careful with hooks, it’s profane in some cases hooks retext-profanities

style-guide/README.md

+              gofmt -s -w api/brokerserver/broker_handler.go
+              ```
+              For Java, just correct the variable name

Contributor

github-actions bot Nov 24, 2025

⚠️ [alex] _{reported by reviewdog 🐶}
just may be insensitive, try not to use it just retext-equality

style-guide/README.md


		```

		To fix them, just execute the command as suggested by the java-formatter. It will auto-format the java sources.

Contributor

github-actions bot Nov 24, 2025

⚠️ [alex] _{reported by reviewdog 🐶}
just may be insensitive, try not to use it just retext-equality

style-guide/README.md


		```

		To fix them, just execute the command as suggested by the java-formatter. It will auto-format the java sources.

Contributor

github-actions bot Nov 24, 2025

⚠️ [alex] _{reported by reviewdog 🐶}
Be careful with execute, it’s profane in some cases execute retext-profanities

style-guide/README.md


		```

		### Skip Pre-Commit Git Hook Locally

Contributor

github-actions bot Nov 24, 2025

⚠️ [alex] _{reported by reviewdog 🐶}
Be careful with Hook, it’s profane in some cases hook retext-profanities

bonzofenix added 3 commits

November 24, 2025 15:57


          Update path in Checkstyle step of java-ci-lint workflow

4d586c0


          Update checkstyle config path in pom.xml to use new relative path

527bf94


          Update path to format_checkstyle.go in java-ci-lint workflow

8e32dda

sonarqubecloud bot commented Nov 24, 2025

Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels