Skip to content

Quick hot fix for compval#128

Merged
ilaflott merged 2 commits intomainfrom
StrictModeTesting
Jul 30, 2025
Merged

Quick hot fix for compval#128
ilaflott merged 2 commits intomainfrom
StrictModeTesting

Conversation

@Ciheim
Copy link
Contributor

@Ciheim Ciheim commented Jul 28, 2025

Fixes path to json file so that the tool can be used with fre-cli

@Ciheim Ciheim requested a review from ceblanton July 28, 2025 21:23
@Ciheim Ciheim self-assigned this Jul 29, 2025
@ilaflott ilaflott self-requested a review July 30, 2025 15:44
Copy link
Member

@ilaflott ilaflott left a comment

Choose a reason for hiding this comment

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

Can catalogbuilder bump the version from 2.0.1 to 2.0.2? or, perhaps, do that in the near-future?

I'd like this so, in the future, the expected fix stays in place where fre-cli expects it, an it doesn't change from under us.

Copy link
Contributor

@ceblanton ceblanton left a comment

Choose a reason for hiding this comment

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

This update looks good, Ciheim. Ian and I strongly believe catalogbuilder needs to embrace more fine-grained versioning to allow fre-cli to test the new updates properly. So let's bump the version for every non-documentation update, even it's a tiny change, to make it a habit here.

@Ciheim
Copy link
Contributor Author

Ciheim commented Jul 30, 2025

My mistake. Moving up to version 2.0.2.

@Ciheim Ciheim requested a review from ilaflott July 30, 2025 19:01
@ilaflott ilaflott merged commit 13b3aa6 into main Jul 30, 2025
4 checks passed
@ilaflott ilaflott deleted the StrictModeTesting branch January 13, 2026 20:08
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.

3 participants