Skip to content
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

[JENKINS-54992] Do not override pronoun on projects only on their scan #150

Merged
merged 1 commit into from
Apr 17, 2019

Conversation

bitwiseman
Copy link
Contributor

@bitwiseman bitwiseman commented Apr 17, 2019

This issue shows that changing the pronoun on a project in
in order to make some actions by the project more clear is
actually problematic.

As reasonable fix, this change makes
the indexing use the source pronoun(s) while reverting the
projects to using their original pronouns.

This PR came from discussions around
jenkinsci/github-branch-source-plugin#195

After this change the Multibranch Pipeline looks like this:
Screen Shot 2019-04-16 at 6 16 36 PM

A github organization folder looks like this:
Screen Shot 2019-04-16 at 6 24 25 PM

This issue shows that changing the pronoun on a project in
in order to make some actions by the project more clear is
actually problematic.  As reasonable fix, this change makes
the indexing use the source pronoun(s) while reverting the
projects to using their original pronouns.
@bitwiseman
Copy link
Contributor Author

@abayer @stephenc @rsandell
So, we can merge this? (I do not have permissions)

@abayer
Copy link
Member

abayer commented Apr 17, 2019

@bitwiseman You do now. =)

@bitwiseman bitwiseman merged commit ac54352 into jenkinsci:master Apr 17, 2019
@bitwiseman bitwiseman deleted the JENKINS-54992 branch April 17, 2019 18:06
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.

4 participants