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

Updated the correct path for resources and main Action class #7877

Closed
wants to merge 1 commit into from

Conversation

MUCCHU
Copy link

@MUCCHU MUCCHU commented Feb 13, 2025

When I followed the tutorial, I found the path for the resources folder and main source files, i.e. the Action and Builder class, was incorrect. I ensured that it is correct so that people who follow it don't face any issue.

@MUCCHU MUCCHU requested a review from a team as a code owner February 13, 2025 18:22
@probot-autolabeler probot-autolabeler bot added the documentation Jenkins documentation, including user and developer docs, solution pages, etc. label Feb 13, 2025
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.

This change is incorrect. Please explain in detail the steps that you took that make you think that this change is needed.

There are many, many examples of the documented pattern for resource access:

@MUCCHU
Copy link
Author

MUCCHU commented Feb 13, 2025

In JetBrains IDE, I see this folder structure
image

@MarkEWaite
Copy link
Contributor

In JetBrains IDE, I see this folder structure

If JetBrains shows "io.jenkins.sample" in the IDE, but stores them on the disc in the "io/jenkins/sample" directory, then that is not a problem. The IDE can show dot separated path names so long as they are stored on the disc with a directory separator ('/' on Unix variants, '\' on Windows)

@MUCCHU
Copy link
Author

MUCCHU commented Feb 13, 2025

Ohh, I got it. I also checked the directory tree and it conforms to your statement(and also the documentation)
image

Sorry for the confusion caused. I just wanted to help.

@MUCCHU MUCCHU closed this Feb 13, 2025
@MUCCHU MUCCHU deleted the patch-1 branch February 13, 2025 19:22
@MarkEWaite
Copy link
Contributor

Sorry for the confusion caused. I just wanted to help.

Thanks for your desire to help! Much appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Jenkins documentation, including user and developer docs, solution pages, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants