-
Notifications
You must be signed in to change notification settings - Fork 376
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
ci(github): Reduce noise from actions labeling #1442
Merged
MichaelsJP
merged 1 commit into
master
from
ci/1440-chore-reduce-the-pr-noise-from-the-conventional-title-action
May 17, 2023
Merged
ci(github): Reduce noise from actions labeling #1442
MichaelsJP
merged 1 commit into
master
from
ci/1440-chore-reduce-the-pr-noise-from-the-conventional-title-action
May 17, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MichaelsJP
force-pushed
the
ci/1440-chore-reduce-the-pr-noise-from-the-conventional-title-action
branch
from
May 17, 2023 08:29
5a4c608
to
d9d8c98
Compare
MichaelsJP
changed the title
ci(github,action): Remove synchronized from the event triggers
ci(github): Remove synchronized from the event triggers
May 17, 2023
MichaelsJP
force-pushed
the
ci/1440-chore-reduce-the-pr-noise-from-the-conventional-title-action
branch
from
May 17, 2023 09:27
4833633
to
f1e3f48
Compare
MichaelsJP
force-pushed
the
ci/1440-chore-reduce-the-pr-noise-from-the-conventional-title-action
branch
from
May 17, 2023 09:28
f1e3f48
to
281e732
Compare
MichaelsJP
force-pushed
the
ci/1440-chore-reduce-the-pr-noise-from-the-conventional-title-action
branch
from
May 17, 2023 09:29
281e732
to
8bbfc02
Compare
MichaelsJP
changed the title
ci(github): Remove synchronized from the event triggers
chore(github): Remove synchronized from the event triggers
May 17, 2023
MichaelsJP
changed the title
chore(github): Remove synchronized from the event triggers
ci(github): Remove synchronized from the event triggers
May 17, 2023
The action are splitted into two now. conventional-commit.yml checks the commits and the title each time somethin happens in the PR. conventional-label.yml is only executed if the PR title is changed, the PR freshly opened, or set to ready for review. This prevents the action from spamming the PR history.
MichaelsJP
force-pushed
the
ci/1440-chore-reduce-the-pr-noise-from-the-conventional-title-action
branch
from
May 17, 2023 11:46
b93080c
to
a80fdf3
Compare
MichaelsJP
changed the title
ci(github): Remove synchronized from the event triggers
ci(github): Reduce noise from actions labeling
May 17, 2023
takb
approved these changes
May 17, 2023
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.
cool
MichaelsJP
deleted the
ci/1440-chore-reduce-the-pr-noise-from-the-conventional-title-action
branch
May 17, 2023 12:35
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This should reduce the PR noise from the conventional label assignment.
Pull Request Checklist
have been resolved.
[Unreleased] heading.
along with a short description of what it is for, and documented this in the Pull Request (below).
(at least Germany), and the graphs build without problems (i.e. no out-of-memory errors).
importer etc.), I have generated longer distance routes for the affected profiles with different options
(avoid features, max weight etc.) and compared these with the routes of the same parameters and start/end
points generated from the current live ORS.
If there are differences then the reasoning for these MUST be documented in the pull request.
and why the change was needed.
Fixes # .
Information about the changes
Examples and reasons for differences between live ORS routes, and those generated from this pull request
Required changes to ors config (if applicable)