Skip to content

Conversation

@pccibot
Copy link
Contributor

@pccibot pccibot commented Aug 12, 2025

modulesync 10.0.0-19-g1adc8e0

@pccibot pccibot added the modulesync PR related to modulesync label Aug 12, 2025
@TheMeier TheMeier changed the title modulesync 10.0.0-19-g1adc8e0 modulesync 10.1.0 Aug 15, 2025
@TheMeier TheMeier added backwards-incompatible This change will lead to a major version bump for the next release and removed modulesync PR related to modulesync labels Aug 15, 2025
@TheMeier TheMeier changed the title modulesync 10.1.0 modulesync 10.1.0 Drop puppet, update openvox minimum version to 8.19 Aug 15, 2025
@TheMeier

This comment was marked as outdated.

@bastelfreak bastelfreak force-pushed the modulesync branch 2 times, most recently from 3e7c5e6 to b5540a2 Compare November 21, 2025 15:02
@bastelfreak bastelfreak added modulesync PR related to modulesync and removed backwards-incompatible This change will lead to a major version bump for the next release labels Nov 21, 2025
@bastelfreak bastelfreak changed the title modulesync 10.1.0 Drop puppet, update openvox minimum version to 8.19 modulesync 10.4.0 Nov 21, 2025
bastelfreak pushed a commit to bastelfreak/puppet-puppetboard that referenced this pull request Dec 5, 2025
the original modulesync PR is failing, so we are trying to apply just
some parts to figure out the broken pieces.

see voxpupuli#431
@smortex

This comment was marked as outdated.

@smortex smortex closed this Dec 26, 2025
@smortex smortex reopened this Dec 26, 2025
@smortex
Copy link
Member

smortex commented Dec 26, 2025

Looks like #437 was never tested and does not pass per:
https://github.com/voxpupuli/puppet-puppetboard/actions/runs/20529046317/job/58977487348?pr=431

Reverting this commit allow the unit tests to pass. #437 has not been released yet, my suggestion is to remove it for now and move on with this module. @bwitt does this look reasonable to you?

@smortex smortex force-pushed the modulesync branch 2 times, most recently from 525e73d to 3656b95 Compare December 26, 2025 21:04
@bwitt
Copy link
Contributor

bwitt commented Dec 26, 2025

@smortex sounds good thanks!

@smortex smortex merged commit 3656b95 into master Dec 26, 2025
10 of 12 checks passed
@smortex smortex deleted the modulesync branch December 26, 2025 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modulesync PR related to modulesync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants