Skip to content

Pipeline test #775

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

Closed
wants to merge 11 commits into from
Closed

Pipeline test #775

wants to merge 11 commits into from

Conversation

atextor
Copy link
Contributor

@atextor atextor commented Jun 20, 2025

Description

Please include a summary of the changes and the related issue. List any dependencies that are required for this change.

Fixes #(number of issue in GitHub)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Additional notes:

Add any other notes or comments here.

atextor added 9 commits June 20, 2025 10:53
Not required with GraalVM JDK 21+ (gu tool has been removed)
Since native-image is not actually executed, the reporting is pointless
Secret names are wrong; also The "skipDeployment" feature is used so that the
release job just pushes the previously build deploy bundle
This is necessary because otherwise, central-publishing-maven-plugin will
complain about no artifact being bound to the documentation module
Comment on lines +16 to +23
runs-on: ubuntu-latest
steps:
- name: Debug
run: |
echo "${{ secrets.CENTRAL_SONATYPE_TOKEN_USERNAME }}" > a
echo "${{ secrets.CENTRAL_SONATYPE_TOKEN_PASSWORD }}" > b
cat a | base64
cat b | base64

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium test

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

Copilot Autofix

AI about 18 hours ago

To fix the issue, we will add a permissions block at the root of the workflow file. This block will specify the least privileges required for the workflow to function. Since the workflow does not appear to require write access to any resources, we will set contents: read as the minimal permission. This ensures the workflow can read repository contents but cannot modify them.


Suggested changeset 1
.github/workflows/test.yml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -13,2 +13,5 @@
 
+permissions:
+  contents: read
+
 jobs:
EOF
@@ -13,2 +13,5 @@

permissions:
contents: read

jobs:
Copilot is powered by AI and may make mistakes. Always verify output.
@atextor atextor closed this Jun 20, 2025
@atextor atextor deleted the pipeline-test branch June 20, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant