- 
                Notifications
    You must be signed in to change notification settings 
- Fork 16
feat: add automated release and labelling #52
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
Conversation
| Yep.  It happened.  The  | 
- [x] introduce PR title conventional commits requirement
- [x] document this in the CONTRIBUTING.md
- [x] add auto labelling workflow and configuration
  - Split from release-drafter since they each are enacted at different times
    - auto-labeler: PR events
    - release-drafter: merge to main
Signed-off-by: jmeridth <jmeridth@gmail.com>
Co-authored-by: Zack Koppert <zkoppert@github.com>
    b637224    to
    c44d657      
    Compare
  
    There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🏷️ and 🤖 , wohoo!
| run: | | ||
| short-tag=$(echo ${{ steps.release-drafter.outputs.tag_name }} | cut -d. -f1) | ||
| echo "SHORT_TAG=$short-tag" >> $GITHUB_OUTPUT | ||
| create_action_images: | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zkoppert this is the new image build and deploy stuff, utilizing the tags generated by release-drafter
Signed-off-by: jmeridth <jmeridth@gmail.com>
3cd024e    to
    1f1d49b      
    Compare
  
    | uses: docker/login-action@v3 | ||
| with: | ||
| registry: ${{ env.REGISTRY }} | ||
| username: ${{ github.actor }} | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Who would github.actor be in this case? github-actions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe so, yes.
Part of github/github-ospo#78
Pull Request
Proposed Changes
packages: writepermissions to actionWe might have a chicken&egg problem where
.github/auto-labeler.ymlhas to be on the default branch before it can be found. I tested this in another personal repo and had that arise.Readiness Checklist
Author/Contributor
make lintand fix any issues that you have introducedmake testand ensure you have test coverage for the lines you are introducingReviewer
bug,documentation,enhancement,infrastructure, orbreaking