Skip to content

(Puppetfile) Switch to our tomcat fork for dependencies#1982

Merged
badenerb merged 1 commit into
productionfrom
IT-6481-Tomcat
Nov 14, 2025
Merged

(Puppetfile) Switch to our tomcat fork for dependencies#1982
badenerb merged 1 commit into
productionfrom
IT-6481-Tomcat

Conversation

@badenerb

@badenerb badenerb commented Nov 12, 2025

Copy link
Copy Markdown
Contributor

Waiting for the official tomcat to approve/merge my PR here: puppetlabs/puppetlabs-tomcat#581

@badenerb badenerb self-assigned this Nov 12, 2025
@badenerb badenerb added the enhancement New feature or request label Nov 12, 2025
@badenerb badenerb marked this pull request as ready for review November 12, 2025 19:26
@badenerb badenerb force-pushed the IT-6481-Tomcat branch 2 times, most recently from 58b8730 to c6a4c77 Compare November 13, 2025 17:39

@jhoblitt jhoblitt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The PR title doesn't state what is being changed to a fork

@badenerb badenerb changed the title (Puppetfile) Switch to our fork for dependancies (Puppetfile) Switch to our tomcat fork for dependancies Nov 13, 2025
@badenerb badenerb requested a review from jhoblitt November 13, 2025 17:47
@jhoblitt jhoblitt changed the title (Puppetfile) Switch to our tomcat fork for dependancies (Puppetfile) Switch to our tomcat fork for dependencies Nov 13, 2025

@jhoblitt jhoblitt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

s/dependancies/dependencies/

Also, adding a link to the upstream PR as a comment in the Puppetfile would be appreciated.

@jhoblitt jhoblitt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@badenerb badenerb merged commit d5dbdab into production Nov 14, 2025
14 checks passed
@badenerb badenerb deleted the IT-6481-Tomcat branch November 14, 2025 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants