Skip to content

Conversation

zshel
Copy link
Contributor

@zshel zshel commented Apr 22, 2025

Instead of matching the whole file with the template, you can now only match a single line and merge the new version into the previous file without overwriting the entire version file.

@AnalogJ
Copy link
Member

AnalogJ commented Apr 22, 2025

hey @zshel !

This looks good, but I'd like to see a passing test included in the PR as well.

@zshel
Copy link
Contributor Author

zshel commented Apr 22, 2025

hey @zshel !

This looks good, but I'd like to see a passing test included in the PR as well.

I don't think the failures have to do with anything I changed, but I'll take a deeper dive.

@AnalogJ
Copy link
Member

AnalogJ commented Apr 23, 2025

yeah don't worry about the failures in the existing tests. Just write a new test for your feature

@zshel
Copy link
Contributor Author

zshel commented Apr 24, 2025

Added a test to cover the new merge option, also added some more tests to cover the general engine as well.

Fixed the CI tests as well, although the chef is still broken, it appears because it's base docker image is so old.

@AnalogJ AnalogJ merged commit c1d41b7 into PackagrIO:master Apr 24, 2025
12 of 13 checks passed
@zshel
Copy link
Contributor Author

zshel commented Apr 25, 2025

Thanks @AnalogJ!

@tnaroska
Copy link

Nice feature. Thanks @zshel . @AnalogJ at some point we should organize a Bumpr user group meet up somewhere. Maybe it's just gonna be the 3 of us, but Bumpr is still being used!! 😄

@zshel
Copy link
Contributor Author

zshel commented Apr 25, 2025

Nice feature. Thanks @zshel . @AnalogJ at some point we should organize a Bumpr user group meet up somewhere. Maybe it's just gonna be the 3 of us, but Bumpr is still being used!! 😄

Ayyyyeee Timo!!!!! Long time no talk, and yes I agree with you, BumprCon 2025! 😁

@AnalogJ
Copy link
Member

AnalogJ commented Apr 25, 2025

hahaha I love this plan. BumprCon NYC? Or should we book a conference center in LA? I'm also open to visiting the BumprCon EU chapter.

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