Skip to content

Basic project validations before attempting to compute code owners - #6

Merged
mzruya merged 1 commit into
mainfrom
mz.validate_project
Apr 3, 2023
Merged

Basic project validations before attempting to compute code owners#6
mzruya merged 1 commit into
mainfrom
mz.validate_project

Conversation

@mzruya

Copy link
Copy Markdown
Contributor

No description provided.

@mzruyamzruya changed the title Perform basic validation on the project structBasic project validations before attempting to compute code ownersApr 3, 2023
@mzruya
mzruyaforce-pushed the mz.validate_project branch from 8c45d12 to 457a82dCompareApril 3, 2023 01:57
@mzruya
mzruya merged commit d398cd2 into mainApr 3, 2023
@mzruya
mzruya deleted the mz.validate_project branch April 10, 2024 19:10
dduugg added a commit that referenced this pull request Aug 17, 2026
Resolves the eight open `actions/missing-workflow-permissions` CodeQL
alerts (#1, #3, #4, #5, #6, #7, #8, #9). Every job now has an effective
`permissions:` block scoped to what it actually does.
ci.yml
Workflow-level `contents: read` covers `check`, `test` and `lints`,
which only check out the repo and run cargo check/test/fmt/clippy.
Four jobs override to `contents: write` because they mutate releases:
- `release` runs `gh release create --generate-notes`
- `upload-mac-universal-bin` and `upload-linux-bin` run
`gh release upload` to attach the built tarballs
- `generate-dotslash-files` uses facebook/dotslash-publish-release,
which also publishes via `gh release upload`
audit.yml
`rustsec/audit-check` needs `checks: write` to publish the advisory
status check on push/pull_request runs and `issues: write` to open an
issue per new advisory on the weekly scheduled run (both documented in
the action's README). `contents: read` covers actions/checkout. It
does not upload SARIF, so no `security-events: write`.
codeql.yml already declares permissions and is untouched.
Sign up for freeto 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

@mzruya