-
Notifications
You must be signed in to change notification settings - Fork 1
Feat: Version strings can now be modified and merged into a complex multiline files. #8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
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. |
yeah don't worry about the failures in the existing tests. Just write a new test for your feature |
…he old chef image
…use of the old chef image" Due to the deprecation of v3 This reverts commit bcbcc7f.
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. |
Thanks @AnalogJ! |
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. |
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.