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.

Sign up for GitHub

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

Jump to bottom

unify internal plan and plan #302

Merged

grosser merged 3 commits into master from grosser/plan

Oct 30, 2023

Owner

grosser commented Oct 27, 2023 •

edited

Loading

also:

making warnings less indirect
sending warnings to stderr
printing planned changes like update will do them (deletes first)
extract plan as file

Checklist

Verified against local install of kennel (using path: in Gemfile)
Added tests

grosser added 3 commits

October 27, 2023 15:51


          remove internal plan vs plan

1c72b3f


          print warnings directly

542d416


          extract plan as class

d937498

grosser merged commit c1112d6 into master

4 checks passed

grosser deleted the grosser/plan branch

October 30, 2023 22:06

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet