8000 Use proper case for GitHub by jsoref · Pull Request #3 · jenkinsci/github-scm-trait-notification-context-plugin · GitHub
[go: up one dir, main page]
Skip to content

Use proper case for GitHub #3

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Sep 24, 2023
Merged

Use proper case for GitHub #3

merged 4 commits into from
Sep 24, 2023

Conversation

jsoref
Copy link
Contributor
@jsoref jsoref commented Mar 11, 2020

@l3ender
Copy link
Contributor
l3ender commented Sep 21, 2023

@steven-foster - are you able to merge PRs in this project, or grant additional access to other contributors? I have a change I'd similarly like to make but am unsure if this repository has any active maintainers.

@l3ender
Copy link
Contributor
l3ender commented Sep 24, 2023

I've obtained maintainer access to this repo so I am moving forward with this PR.

One thing I notice is that updating the URL to https://wiki.jenkins-ci.org/display/JENKINS/GitHub+Custom+Notification+Context+SCM+Behaviour (instead of https://wiki.jenkins-ci.org/display/JENKINS/Github+Custom+Notification+Context+SCM+Behaviour) causes a redirect to Google instead of a redirect to the plugin documentation. I believe this will be resolved when a new release occurs for the plugin and the plugin documentation site is updated (I plan to do before long).

Thanks for your contribution!

@l3ender l3ender added the fix Updating something that isn't a bug label Sep 24, 2023
@l3ender l3ender merged commit 36632d0 into jenkinsci:master Sep 24, 2023
@jsoref jsoref deleted the github branch September 24, 2023 19:00
@l3ender
Copy link
Contributor
l3ender commented Sep 24, 2023

Released in 28.v36632d058d08. Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@l3ender l3ender l3ender approved these changes

Assignees
No one assigned
Labels
fix Updating something that isn't a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jsoref @l3ender
0