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

Update readme to highlight interaction between "Authenticated" user group and Permission checking #210

Merged
merged 3 commits into from
Sep 9, 2023

Conversation

fchebbo
Copy link

@fchebbo fchebbo commented Sep 8, 2023

This pull request adds a minor update to the readme.

The reason I am adding this, is that recently we ran into a issue where we were suddenly seeing the

Unable to find project for artifact copy: YOUR_PROJECT_WITH_ARTIFACTS
This may be due to incorrect project name or permission settings; see help for project name in job configuration.
Build step 'Copy artifacts from another project' marked build as failure

message after a Jenkins Update and couldn't figure out why. It was only after looking at the source code (in particular here) that I was able to determine the issue.

This PR is a suggestion to include this additional line in the readme, as I don't believe that one should need to look into the source code to determine such a behavior.

Testing done

Ensured the readme looks "good" on the github page. This PR contains no actual code change

Submitter checklist

Preview Give feedback

@fchebbo fchebbo requested a review from a team as a code owner September 8, 2023 23:12
Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks very much! Alternate phrasing that is optional. Change is accepted whether you take the suggestion or not.

README.adoc Outdated Show resolved Hide resolved
Co-authored-by: Mark Waite <mark.earl.waite@gmail.com>
@MarkEWaite MarkEWaite enabled auto-merge (squash) September 8, 2023 23:36
@MarkEWaite MarkEWaite merged commit 6fdb370 into jenkinsci:master Sep 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants