Skip to content

feat: add status command#10

Merged
lukeed merged 4 commits intomasterfrom
feat/status
Sep 23, 2020
Merged

feat: add status command#10
lukeed merged 4 commits intomasterfrom
feat/status

Conversation

@lukeed
Copy link
Copy Markdown
Owner

@lukeed lukeed commented Sep 23, 2020

The status command (available via both CLI and API) checks for outstanding migration files.
In other words, how many up migrations have not yet been applied.

The following screenshots are taken with an empty database as my starting point:

Note: The node bin is identical to ley command... local dev.

Screen Shot 2020-09-23 at 10 49 16 AM

Screen Shot 2020-09-23 at 10 49 25 AM


I also cleaned up the README a bit. Repeating the same opts.cwd, ..., opts.cwd was bugging me.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #10 into master will decrease coverage by 2.13%.
The diff coverage is 14.28%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   48.07%   45.94%   -2.14%     
==========================================
  Files           2        2              
  Lines         104      111       +7     
==========================================
+ Hits           50       51       +1     
- Misses         54       60       +6     
Impacted Files Coverage Δ
index.js 11.76% <14.28%> (+0.28%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c97dc45...14bb525. Read the comment docs.

@lukeed
Copy link
Copy Markdown
Owner Author

lukeed commented Sep 23, 2020

/cc @TehShrike

@lukeed lukeed merged commit ad38a53 into master Sep 23, 2020
@lukeed lukeed deleted the feat/status branch September 23, 2020 18:25
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.

2 participants