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 parent pom #105

Closed
wants to merge 6 commits into from
Closed

Update parent pom #105

wants to merge 6 commits into from

Conversation

devc007
Copy link

@devc007 devc007 commented Dec 25, 2023

allow compilation with java 11

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did

@devc007 devc007 requested a review from a team as a code owner December 25, 2023 23:40
@devc007
Copy link
Author

devc007 commented Dec 27, 2023

Help Needed

I have tried everything but I couldn't figure out how to pass all the checks.

pom.xml Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
pom.xml Outdated
@@ -70,7 +71,7 @@
<dependencies>
<dependency>
<groupId>io.jenkins.tools.bom</groupId>
<artifactId>bom-2.332.x</artifactId>
<artifactId>bom-2.401.x</artifactId>
<version>1466.v85a_616ea_b_87c</version>
Copy link
Member

Choose a reason for hiding this comment

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

If you're changing the artifactId you'll also need to change the version to one that exists. Your current build error is artifact not found.

Copy link

@MarkEWaite MarkEWaite Jan 1, 2024

Choose a reason for hiding this comment

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

List of bill of materials releases by version are available from the BOM releases page for active versions and in the "Depending on older versions" table for inactive versions

The "Improve a plugin" tutorial that updates the base Jenkins version includes guidance on this step.

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for pointing this mistake/

@devc007
Copy link
Author

devc007 commented Jan 1, 2024

Do I need to change more in pom.xml file like changing tha packaging type?

@MarkEWaite
Copy link

Do I need to change more in pom.xml file like changing the packaging type?

No. The changes you've made are correct and complete. The Jenkinsfile that is run by ci.jenkins.io is from the master branch, nof from your pull request. I used my admin privileges on ci.jenkins.io to run the job with your Jenkinsfile.

@devc007
Copy link
Author

devc007 commented Jan 2, 2024

Mark can you tell me what should i do in my upcoming Pull request so that all the checks will pass in first go? Is this possible?

@MarkEWaite
Copy link

Mark can you tell me what should i do in my upcoming Pull request so that all the checks will pass in first go? Is this possible?

If you are changing the Jenkinsfile and are not a maintainer of the plugin, then your changes will not be executed by ci.jenkins.io because we don't want the ci.jenkins.io executors to be misused by malicious users submitting changes to a Jenkinsfile

@devc007 devc007 closed this by deleting the head repository Sep 24, 2024
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.

5 participants