Skip to content
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

Set permissions on formatter workflow #13227

Merged
merged 2 commits into from
Feb 5, 2024

Conversation

jeskew
Copy link
Contributor

@jeskew jeskew commented Feb 5, 2024

Granting the formatting workflow explicit write permissions on repo contents and issues to address https://github.com/Azure/bicep/actions/runs/7771672951/job/21193245862#step:7:27

Rerunning the workflow with the changes from this PR succeeded, resulting in the a8de84f commit included in this PR.

Microsoft Reviewers: Open in CodeFlow

@jeskew jeskew requested a review from a team February 5, 2024 15:22
Copy link
Contributor

github-actions bot commented Feb 5, 2024

Test this change out locally with the following install scripts (Action run 7787034499)

VSCode
  • Mac/Linux
    bash <(curl -Ls https://aka.ms/bicep/nightly-vsix.sh) --run-id 7787034499
  • Windows
    iex "& { $(irm https://aka.ms/bicep/nightly-vsix.ps1) } -RunId 7787034499"
Azure CLI
  • Mac/Linux
    bash <(curl -Ls https://aka.ms/bicep/nightly-cli.sh) --run-id 7787034499
  • Windows
    iex "& { $(irm https://aka.ms/bicep/nightly-cli.ps1) } -RunId 7787034499"

Copy link
Contributor

github-actions bot commented Feb 5, 2024

Test Results

    66 files   -     31      66 suites   - 31   24m 34s ⏱️ - 28m 37s
11 009 tests  -     20  11 009 ✅  -     20  0 💤 ±0  0 ❌ ±0 
25 908 runs   - 12 861  25 908 ✅  - 12 861  0 💤 ±0  0 ❌ ±0 

Results for commit a8de84f. ± Comparison against base commit 624095f.

@jeskew jeskew enabled auto-merge (squash) February 5, 2024 15:34
@jeskew jeskew merged commit f8a29d2 into main Feb 5, 2024
44 checks passed
@jeskew jeskew deleted the jeskew/set-permissions-on-formatter-workflow branch February 5, 2024 15:41
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.

None yet

2 participants