Skip to content

feat: rie private to public automation initial merge (#172)#174

Merged
tobixlea merged 1 commit intomainfrom
develop
Mar 30, 2026
Merged

feat: rie private to public automation initial merge (#172)#174
tobixlea merged 1 commit intomainfrom
develop

Conversation

@tobixlea
Copy link
Copy Markdown
Contributor

  • Update from upstream - 2026-03-18

  • Update from upstream - 2026-03-18

  • Update from upstream - 2026-03-18

  • Update from upstream - 2026-03-19

  • Update from upstream - 2026-03-19

  • Potential fix for code scanning alert no. 7: Workflow does not contain permissions

  • Update from upstream - 2026-03-27

  • Update from upstream - 2026-03-30


Issue #, if available:

Description of changes:
Merging changes from develop branch to main branch

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

* Update from upstream - 2026-03-18

* Update from upstream - 2026-03-18

* Update from upstream - 2026-03-18

* Update from upstream - 2026-03-19

* Update from upstream - 2026-03-19

* Potential fix for code scanning alert no. 7: Workflow does not contain permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

* Update from upstream - 2026-03-27

* Update from upstream - 2026-03-30

---------

Co-authored-by: lambda-tooling+rie <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@tobixlea tobixlea requested a review from a team March 30, 2026 18:42
Comment on lines +10 to +20
runs-on: ubuntu-latest
steps:
- name: Check source branch
run: |
SOURCE_BRANCH="${{ github.head_ref }}"
if [[ "$SOURCE_BRANCH" != "develop" ]]; then
echo "Error: Only pull requests from develop branch are allowed into main"
echo "Current source branch ($SOURCE_BRANCH)."
exit 1
fi
echo "Source branch is develop - merge allowed"

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

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 4 days ago

In general, this problem is fixed by explicitly defining a permissions: block for the workflow or for individual jobs, granting only the minimal scopes required. For a job that only reads context information (like github.head_ref) and doesn’t interact with repository contents, PRs, or issues, the least‑privilege configuration is to set contents: read at the workflow or job level (or even permissions: {} if no GitHub API access is needed, but contents: read is a common safe baseline).

For this specific file, the simplest non‑breaking fix is to add a root‑level permissions: block that applies to all jobs. The workflow only reads metadata and performs shell logic, so we can restrict the token to read‑only repository contents. Concretely, in .github/workflows/validate-branch-into-main.yaml, insert:

permissions:
  contents: read

between the name: section and the on: block (e.g., after line 2), leaving all job and step definitions unchanged. No imports or additional methods are needed since this is just a YAML configuration change and does not affect the workflow’s behavior.

Suggested changeset 1
.github/workflows/validate-branch-into-main.yaml

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/validate-branch-into-main.yaml b/.github/workflows/validate-branch-into-main.yaml
--- a/.github/workflows/validate-branch-into-main.yaml
+++ b/.github/workflows/validate-branch-into-main.yaml
@@ -1,5 +1,8 @@
 name: Validate PR Branch into Main 
 
+permissions:
+  contents: read
+
 on:
   pull_request:
     branches:
EOF
@@ -1,5 +1,8 @@
name: Validate PR Branch into Main

permissions:
contents: read

on:
pull_request:
branches:
Copilot is powered by AI and may make mistakes. Always verify output.
@tobixlea tobixlea merged commit eaf70a7 into main Mar 30, 2026
10 of 13 checks passed
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.

4 participants