Skip to content

Commit

Permalink
Split parsing logic into separate crate (#83)
Browse files Browse the repository at this point in the history
* Split mask parsing logic into separate workspace crate

* Fix paths

* Move maskfile with build commands into mask

* Update tests

* Add test cmd for mask-parser

* Cleanup imports

* Add test cmd for entire workspace

* Revert back to single root maskfile

* Fix link path

* Fix maskfile path in test

* Update deps

* Expose parse method from parser

* Return a Maskfile instead of a root Command

The root command wasn't really a Command... it was just reusing that struct for simplicity.

Now a full Maskfile definition is returned.

* Force colored to output colors in CI environment

* Make the Maskfile struct serializable to json

* Make cmd.script optional

Parent commands may not have a script, but their subcommands do. Making script optional makes a lot more sense

* Prep mask-parser for publishing

* Remove unnecessary exclusion list

* Limit to maximum of 5 keywords

* Remove old mask lib

Don't need this anymore now that we have the mask-parser crate published.

* Note mask-parser is published
  • Loading branch information
jacobdeichert authored Apr 15, 2021
1 parent 3cd5353 commit 6227bef
Show file tree
Hide file tree
Showing 21 changed files with 543 additions and 653 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@ jobs:
fail-fast: false
matrix:
platform: [ubuntu-latest, windows-latest, macos-latest]
env:
CLICOLOR_FORCE: 1
steps:
- name: Checkout master
uses: actions/checkout@master
Expand Down
Loading

0 comments on commit 6227bef

Please sign in to comment.