Skip to content
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

Refactor the Plan printing functionality #8

Merged
merged 1 commit into from
Apr 25, 2023

Conversation

marchinho11
Copy link
Owner

No description provided.

@marchinho11 marchinho11 added the bug Something isn't working label Apr 23, 2023
@marchinho11 marchinho11 self-assigned this Apr 23, 2023
@codecov
Copy link

codecov bot commented Apr 23, 2023

Codecov Report

Merging #8 (7804b8e) into release/0.0.6 (8289bae) will increase coverage by 2.27%.
The diff coverage is 95.34%.

❗ Current head 7804b8e differs from pull request most recent head 3230fae. Consider uploading reports for the commit 3230fae to get more accurate results

@@                Coverage Diff                @@
##           release/0.0.6       #8      +/-   ##
=================================================
+ Coverage          95.31%   97.59%   +2.27%     
=================================================
  Files                 32       33       +1     
  Lines                854      872      +18     
=================================================
+ Hits                 814      851      +37     
+ Misses                40       21      -19     
Impacted Files Coverage Δ
hnhm/plan_printer.py 95.23% <95.23%> (ø)
hnhm/cli.py 97.22% <100.00%> (+20.75%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@marchinho11 marchinho11 force-pushed the feature/refactor-plan-printer branch from 7804b8e to 3230fae Compare April 25, 2023 19:03
@marchinho11 marchinho11 merged commit c1553b7 into release/0.0.6 Apr 25, 2023
@marchinho11 marchinho11 deleted the feature/refactor-plan-printer branch April 25, 2023 19:05
marchinho11 added a commit that referenced this pull request Apr 27, 2023
* Refactor the Plan printing functionality (#8)

* Internal improvements (#9)

* Check all Group attributes are mapped in a Flow

* Use the `.get` function in `to_core` functions

* Rename "Mutation" to "Migration"

* Remove assertions from the `plan` function

* Up package version to 0.0.6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant