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

feat(cli): add compas watch [..sub-commands] and --watch #1587

Merged
merged 7 commits into from Feb 9, 2022

Conversation

dirkdev98
Copy link
Member

@dirkdev98 dirkdev98 commented Feb 6, 2022

Closes #1509

TODO:

  • Document watch behavior + longDescription
  • Document modifiers.isWatchable and watchSettings
    • And that you need to use 'extending the cli' in compas run if you need custom watch settings
  • Document config/compas.json and expected structure, and that it's optional in most cases
  • Sync metadata, so cli reference is regenerated

@codecov
Copy link

codecov bot commented Feb 6, 2022

Codecov Report

Merging #1587 (4547632) into main (5a2e6bf) will decrease coverage by 0.64%.
The diff coverage is 46.98%.

❗ Current head 4547632 differs from pull request most recent head 97e6da2. Consider uploading reports for the commit 97e6da2 to get more accurate results

@@            Coverage Diff             @@
##             main    #1587      +/-   ##
==========================================
- Coverage   76.20%   75.56%   -0.65%     
==========================================
  Files         162      164       +2     
  Lines       25772    26343     +571     
==========================================
+ Hits        19639    19905     +266     
- Misses       6133     6438     +305     

@dirkdev98 dirkdev98 enabled auto-merge (squash) February 9, 2022 16:23
@dirkdev98 dirkdev98 merged commit b1f0762 into main Feb 9, 2022
@dirkdev98 dirkdev98 deleted the feat/cli-watch branch February 9, 2022 16:24
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.

cli(cli): add support to watch commands
1 participant