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

New pipeline #1070

Open
wants to merge 3 commits into
base: 2.x
Choose a base branch
from
Open

Conversation

Spomky
Copy link
Contributor

@Spomky Spomky commented Sep 19, 2022

This PR is a proposal for a new pipeline management.
It is derived from what is done with e.g. web-token or webauthn.

  • Github Community Standards complementary files
    • CONTRIBUTING: .github/CONTRIBUTING.md file
    • Pull request template: .github/PULL_REQUEST_TEMPLATE.md
    • New issue templates
  • dependabot rules
  • stale rules
  • Makefile in favor of composer scripts
  • Pipeline:
    • .github/workflows/merge-me.yml : auto merge PR from dependabot (depends on CI status)
    • .github/workflows/release-on-milestone-closed.yml : this GH Action is very useful and allows seamless branches and tags mgmt.
      • Branches shall be e.g. 2.0.x, 2.1.x, 3.0.x...
      • Milestones shall have the same version numbering, but x is an actual number (e.g. 2.0.1, 2.0.2, 2.1.0...).
      • When a milestone is closed (e.g. 2.7.0), this action will add a tag 2.7.0 on the branch, create minor and minor branches if deemed necessary (2.8.x and 3.0.x) and create a PR for merging 2.7.0 to 2.8.x.
      • All PR and issues should point to the correct milestone. This will help creating nice release notes such as this one: https://github.com/web-auth/webauthn-framework/releases/tag/4.2.0

Please note that some files require changes to be in line with this project.
In addition, the GH Actions require some project or organization secrets:

  • "GITHUB_TOKEN": ${{ secrets.GITHUB_TOKEN }}
  • "SIGNING_SECRET_KEY": ${{ secrets.SIGNING_SECRET_KEY }}
  • "GIT_AUTHOR_NAME": ${{ secrets.GIT_AUTHOR_NAME }}
  • "GIT_AUTHOR_EMAIL": ${{ secrets.GIT_AUTHOR_EMAIL }}

To be discussed.

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.

1 participant