Title: write permission is conditionally needed by release-drafter · Issue #47 · cpp-linter/.github · GitHub
Open Graph Title: write permission is conditionally needed by release-drafter · Issue #47 · cpp-linter/.github
X Title: write permission is conditionally needed by release-drafter · Issue #47 · cpp-linter/.github
Description: The cpp-linter repo has 2 workflows that call into the reusable workflow, .github/workflows/release-drafter.yml: cpp-linter/cpp-linter/.github/workflows/labeler.yml requires permissions: contents: read pull-requests: write cpp-linter/cpp...
Open Graph Description: The cpp-linter repo has 2 workflows that call into the reusable workflow, .github/workflows/release-drafter.yml: cpp-linter/cpp-linter/.github/workflows/labeler.yml requires permissions: contents: ...
X Description: The cpp-linter repo has 2 workflows that call into the reusable workflow, .github/workflows/release-drafter.yml: cpp-linter/cpp-linter/.github/workflows/labeler.yml requires permissions: contents: ...
Opengraph URL: https://github.com/cpp-linter/.github/issues/47
X: @github
Domain: patch-diff.githubusercontent.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"write permission is conditionally needed by release-drafter","articleBody":"The cpp-linter repo has 2 workflows that call into the reusable workflow, `.github/workflows/release-drafter.yml`:\n1. cpp-linter/cpp-linter/.github/workflows/labeler.yml requires\n ```yml\n permissions:\n contents: read\n pull-requests: write\n ```\n2. cpp-linter/cpp-linter/.github/workflows/release-drafter.yml; requires\n ```yml\n permissions:\n contents: write\n pull-requests: read\n ```\n\nWhile the reusable workflow demands write permission regardless of calling context: \nhttps://github.com/cpp-linter/.github/blob/86b6518d4136a9a82ea8c9f040d282efd9c88313/.github/workflows/release-drafter.yml#L15-L19\n\n## Proposal\n\nThe `permissions` map cannot be dynamically set.\n\n```yml\n# THIS WILL NOT WORK!\npermissions:\n contents: ${{ inputs.mk-labels \u0026\u0026 'read' || 'write' }}\n pull-requests: ${{ inputs.mk-draft-release \u0026\u0026 'read' || 'write' }}\n```\nThis means we need 2 separate reusable workflows for each calling context:\n- To auto-label PRs\n- To draft (or update a drafted) release\n\nBecause the reusable workflow is just 1 step, I'm inclined to just copy that 1 step into the calling workflows' yml and modify the permissions as needed.\n\n### Or use [git-cliff] + [gh-cli]\n\nIn the cpp-linter-rs repo, I use a script (written in python) to\n\n1. Bump the version according to `inputs.component`. In rust this requires altering the Cargo.toml manifest. In python we would only need to calculate the version based on the last tag.\n2. Use [git-cliff] with the new version number to generate release notes (used in the final step 4). This step also regenerates the entire CHANGELOG.md in repo root.\n3. Use `git` to push any metadata file changes. The step is not required where `setuptools-scm` automatically sets the package version based on `git describe --tags` (or similar).\n4. Use [gh-cli] (requires a PAT with `permissions: {contents: write}`) to publish a release, which inherently creates a new tag that triggers build/deploy CI workflows.\n\n\u003e [!caution]\n\u003e step 4 requires a PAT instead of the generic/default `secrets.GITHUB_TOKEN` because CI triggers are ignored for events caused by \"github-actions[bot]\".\n\nThis might seem overly complicated, but it provides much better flexibility and does not surrender security to third-party software that we don't control/understand.\n\n[gh-cli]: https://cli.github.com/\n[git-cliff]: https://git-cliff.org/\n","author":{"url":"https://github.com/2bndy5","@type":"Person","name":"2bndy5"},"datePublished":"2025-09-04T12:47:04.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":17},"url":"https://github.com/47/.github/issues/47"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:11d43754-ff9b-bc84-0341-6731bdd94b25 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | B540:1A4E67:1F509E:2C6E4D:696F2D7C |
| html-safe-nonce | 2ff2b49fe486d581768014bbdbfa8891f905c05da52d208350cbcd268b1e7f82 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCNTQwOjFBNEU2NzoxRjUwOUU6MkM2RTREOjY5NkYyRDdDIiwidmlzaXRvcl9pZCI6IjUwNzM1MjM4NzA3MTUwMzA5MDgiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 620199f212d95243fd21ee72c0964aa968a98ffe15ba6ba57a3e4f12ed8de954 |
| hovercard-subject-tag | issue:3383626288 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/cpp-linter/.github/47/issue_layout |
| twitter:image | https://opengraph.githubassets.com/9eae4e45d7a7d87b545e47da2d2fd8ce0bd32eb1a58cf9da670a5cd98b2a8b05/cpp-linter/.github/issues/47 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/9eae4e45d7a7d87b545e47da2d2fd8ce0bd32eb1a58cf9da670a5cd98b2a8b05/cpp-linter/.github/issues/47 |
| og:image:alt | The cpp-linter repo has 2 workflows that call into the reusable workflow, .github/workflows/release-drafter.yml: cpp-linter/cpp-linter/.github/workflows/labeler.yml requires permissions: contents: ... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | 2bndy5 |
| hostname | github.com |
| expected-hostname | github.com |
| None | b278ad162d35332b6de714dfb005de04386c4d92df6475522bef910f491a35ee |
| turbo-cache-control | no-preview |
| go-import | github.com/cpp-linter/.github git https://github.com/cpp-linter/.github.git |
| octolytics-dimension-user_id | 103884627 |
| octolytics-dimension-user_login | cpp-linter |
| octolytics-dimension-repository_id | 513759552 |
| octolytics-dimension-repository_nwo | cpp-linter/.github |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 513759552 |
| octolytics-dimension-repository_network_root_nwo | cpp-linter/.github |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 39aed5006635ab6f45e6b77d23e73b08a00272a3 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width