-
Notifications
You must be signed in to change notification settings - Fork 7
updates on ci and add dependabot config #16
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
| types: | ||
| - opened | ||
| - reopened | ||
| - synchronize |
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 would recommend reverting this, otherwise it will run any time there is any kind of update to a PR (e.g. title changes, it gets merged, someone comments, etc.)
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.
That does not happen, but i will revert
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.
Yep these would only trigger on a push to the branch and when the PR is (re)opened.
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.
Normally, it will trigger when you open/reopen/sync. If you want to trigger other actions, then you set it.
so that is redundant to set as those are the default triggers
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.
@cpanato Huh I didn't know that, they really didn't make it easy to find. Learn something new every day!
From the first paragraph under the notes here:
Runs your workflow when activity on a pull request in the workflow's repository occurs. For example, if no activity types are specified, the workflow runs when a pull request is opened or reopened or when the head branch of the pull request is updated.
Then again, to quote the Zen of Python, explicit is better than implicit 😄
Signed-off-by: cpanato <ctadeu@gmail.com>
No description provided.