ci, doc: Document required actions #1408

pull hebasto wants to merge 1 commits into bitcoin-core:master from hebasto:230821-gha-doc changing 1 files +7 −0
  1. hebasto commented at 9:50 pm on August 21, 2023: member

    Suggested in https://github.com/bitcoin/bitcoin/pull/28173#pullrequestreview-1587098202:

    I wonder if it makes sense to document the required permissions inside the file?

  2. ci, doc: Document required actions 1d29b8426a
  3. real-or-random commented at 8:53 am on August 22, 2023: contributor

    Hm, isn’t this clear from reading the files? I think that means that adding a comment is mostly an additional maintenance burden because we need to keep it up-to-date (and no tool complains if we forget).

    Also, the required permissions are not canonical. You could also allow any action, or pin the actions to specific commit ids. All of these options would work.

  4. real-or-random added the label ci on Aug 22, 2023
  5. hebasto closed this on Aug 22, 2023


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/secp256k1. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-10-31 23:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me