Skip to content

Do not generate update-center.json.html anymore#889

Open
daniel-beck wants to merge 1 commit intojenkins-infra:masterfrom
daniel-beck:remove-update-center.json.html
Open

Do not generate update-center.json.html anymore#889
daniel-beck wants to merge 1 commit intojenkins-infra:masterfrom
daniel-beck:remove-update-center.json.html

Conversation

@daniel-beck
Copy link
Contributor

Consuming this file was removed in jenkinsci/jenkins#3970, so it's probably time to clean up here.

@daniel-beck daniel-beck added enhancement This is an enhancement for the tool or wrapper scripts, typically adding features. java This PR affects Java code and would need a new release to be effective labels Sep 24, 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.

Thanks!

@dduportal
Copy link
Contributor

Looks good! Can you hold the merge of this one until we've fully migrated the publick8s cluster ? (ref. jenkins-infra/helpdesk#4617 (comment))

@daniel-beck daniel-beck added the wrapper PR affects wrapper behavior and will be effective immediately when merged. label Sep 25, 2025
Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

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

  • All is good on infra side to support this change
  • LGTM for code (checked locally the site generation with success)

you can proceed whenever you want @daniel-beck

@MarkEWaite
Copy link
Contributor

Per the comment from @dduportal, this is ready to merge

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

Labels

enhancement This is an enhancement for the tool or wrapper scripts, typically adding features. java This PR affects Java code and would need a new release to be effective wrapper PR affects wrapper behavior and will be effective immediately when merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants