Skip to content

chore: Add title to Merge Notification CI job#1049

Merged
chiedo merged 1 commit intogithub:mainfrom
nschonni:merge-notification-job-name
Nov 5, 2020
Merged

chore: Add title to Merge Notification CI job#1049
chiedo merged 1 commit intogithub:mainfrom
nschonni:merge-notification-job-name

Conversation

@nschonni
Copy link
Copy Markdown
Contributor

@nschonni nschonni commented Nov 4, 2020

Why:

On the jobs page, because the workflow doesn't have a name, it just shows as the file name

image

What's being changed:

Add a name value so that the job will be rendered under that string instead of the file name. Also swapped the testing from testing that if the job is running on a private repo, to compare to the repository name like the other jobs

Check off the following:

@nschonni nschonni requested a review from a team as a code owner November 4, 2020 00:12
@nschonni nschonni force-pushed the merge-notification-job-name branch from 550e978 to 8a17d40 Compare November 5, 2020 19:27
Also switch to repository filtering like other jobs
@nschonni nschonni force-pushed the merge-notification-job-name branch from 8a17d40 to 42ae9e3 Compare November 5, 2020 19:35
@nschonni
Copy link
Copy Markdown
Contributor Author

nschonni commented Nov 5, 2020

@chiedo I think this one is pretty straight forward fix, for a minor annoyance 😄

@chiedo chiedo merged commit 1da772f into github:main Nov 5, 2020
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 5, 2020

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours.

@nschonni nschonni deleted the merge-notification-job-name branch November 5, 2020 19:58
jnidzwetzki pushed a commit to jnidzwetzki/docs that referenced this pull request Oct 6, 2022
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.

2 participants