-
Notifications
You must be signed in to change notification settings - Fork 36
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
Reusable workflows #1278
Reusable workflows #1278
Conversation
be924fc
to
08d7b86
Compare
Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
Signed-off-by: Nikita Skrynnik <nikita.skrynnik@xored.com>
c29e89d
to
7e84daa
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.
LGTM 🚀
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
MAXIMUM_RETRIES: 25 | ||
if: ${{ github.event.workflow_run.conclusion == 'success' && github.actor == 'nsmbot'}} | ||
uses: networkservicemesh/.github/.github/workflows/automerge.yaml@main |
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.
Do we want to depend on a branch here rather than a specific point?
I can see tradeoffs either way... so the real question is 'have we thought it through' ?
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.
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.
Sounds good :)
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.
@edwarnicke I've created a ticket for v1.5.0 networkservicemesh/.github#6
Overall other than my branch vs commit comment above, I'm good with this :) |
Description
Issue link
networkservicemesh/.github#4
How Has This Been Tested?
Types of changes