Skip to content

Commit

Permalink
Fix CI
Browse files Browse the repository at this point in the history
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
  • Loading branch information
ggwpez committed Aug 15, 2023
1 parent 5abb781 commit 99427c2
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 5 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,18 @@ jobs:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v3
with:
fetch-depth: 1

- name: Cargo install
run: cargo install --git ${{ github.server_url }}/${{ github.repository }} zepter --rev $GITHUB_SHA --locked -q
run: cargo install --path . --locked -q

- name: CLI works
run: zepter --version && zepter --help

- name: Cargo install (no-default-features)
run: cargo install --git ${{ github.server_url }}/${{ github.repository }} zepter --rev $GITHUB_SHA --locked -q --no-default-features
run: cargo install --path . --locked -q --no-default-features

- name: CLI works (no-default-features)
run: zepter --version && zepter --help
6 changes: 3 additions & 3 deletions tests/ui/root-args/help.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ cases:
Options:
--quiet Only print errors. Supersedes `--log`
--log <LEVEL> Log level to use [default: info]
--color Do use ANSI terminal colors
--color Use ANSI terminal colors
-h, --help Print help
-V, --version Print version
- cmd: lint --help
Expand All @@ -34,7 +34,7 @@ cases:
Options:
--quiet Only print errors. Supersedes `--log`
--log <LEVEL> Log level to use [default: info]
--color Do use ANSI terminal colors
--color Use ANSI terminal colors
-h, --help Print help
- cmd: lint propagate-feature --help
stdout: "Check whether features are properly propagated\n\nUsage: zepter lint propagate-feature [OPTIONS] --feature <FEATURE>\n\nOptions:\n --manifest-path <MANIFEST_PATH>\n Cargo manifest path or directory.\n \n For directories it appends a `Cargo.toml`.\n \n [default: Cargo.toml]\n\n --workspace\n Whether to only consider workspace crates\n\n --offline\n Whether to use offline mode\n\n --quiet\n Only print errors. Supersedes `--log`\n\n --locked\n Whether to use all the locked dependencies from the `Cargo.lock`.\n \n Otherwise it may update some dependencies. For CI usage its a good idea to use it.\n\n --log <LEVEL>\n Log level to use\n \n [default: info]\n\n --all-features\n \n\n --color\n Do use ANSI terminal colors\n\n --feature <FEATURE>\n The feature to check\n\n -p, --packages [<PACKAGES>...]\n The packages to check. If empty, all packages are checked\n\n --feature-enables-dep <FEATURE:CRATE>\n The auto-fixer will enables the feature of the dependencies as non-optional.\n \n This can be used in case that a dependency should not be enabled like `dep?/feature` but\n like `dep/feature` instead. In this case you would pass `--feature-enables-dep\n feature:dep`. The option can be passed multiple times, or multiple key-value pairs can be\n passed at once by separating them with a comma like: `--feature-enables-dep\n feature:dep,feature2:dep2`. (TODO: Duplicate entries are undefined).\n\n --left-side-feature-missing <MUTE_SETTING>\n Overwrite the behaviour when the left side dependency is missing the feature.\n \n This can be used to ignore missing features, treat them as warning or error. A \"missing\n feature\" here means that if `A` has a dependency `B` which has a feature `F`, and the\n propagation is checked then normally it would error if `A` is not forwarding `F` to `B`.\n Now this option modifies the behaviour if `A` does not have the feature in the first place.\n The default behaviour is to require `A` to also have `F`.\n \n [default: error]\n\n Possible values:\n - ignore: Ignore this behaviour\n - error: Treat as error\n\n --show-version\n Show crate versions in the output\n\n --fix\n Try to automatically fix the problems\n\n --modify-paths <MODIFY_PATHS>\n \n\n --fix-dependency <FIX_DEPENDENCY>\n Fix only issues with this package as dependency\n\n --fix-package <FIX_PACKAGE>\n Fix only issues with this package as feature source\n\n -h, --help\n Print help (see a summary with '-h')\n"
stdout: "Check whether features are properly propagated\n\nUsage: zepter lint propagate-feature [OPTIONS] --feature <FEATURE>\n\nOptions:\n --manifest-path <MANIFEST_PATH>\n Cargo manifest path or directory.\n \n For directories it appends a `Cargo.toml`.\n \n [default: Cargo.toml]\n\n --workspace\n Whether to only consider workspace crates\n\n --offline\n Whether to use offline mode\n\n --quiet\n Only print errors. Supersedes `--log`\n\n --locked\n Whether to use all the locked dependencies from the `Cargo.lock`.\n \n Otherwise it may update some dependencies. For CI usage its a good idea to use it.\n\n --log <LEVEL>\n Log level to use\n \n [default: info]\n\n --all-features\n \n\n --color\n Use ANSI terminal colors\n\n --feature <FEATURE>\n The feature to check\n\n -p, --packages [<PACKAGES>...]\n The packages to check. If empty, all packages are checked\n\n --feature-enables-dep <FEATURE:CRATE>\n The auto-fixer will enables the feature of the dependencies as non-optional.\n \n This can be used in case that a dependency should not be enabled like `dep?/feature` but\n like `dep/feature` instead. In this case you would pass `--feature-enables-dep\n feature:dep`. The option can be passed multiple times, or multiple key-value pairs can be\n passed at once by separating them with a comma like: `--feature-enables-dep\n feature:dep,feature2:dep2`. (TODO: Duplicate entries are undefined).\n\n --left-side-feature-missing <MUTE_SETTING>\n Overwrite the behaviour when the left side dependency is missing the feature.\n \n This can be used to ignore missing features, treat them as warning or error. A \"missing\n feature\" here means that if `A` has a dependency `B` which has a feature `F`, and the\n propagation is checked then normally it would error if `A` is not forwarding `F` to `B`.\n Now this option modifies the behaviour if `A` does not have the feature in the first place.\n The default behaviour is to require `A` to also have `F`.\n \n [default: error]\n\n Possible values:\n - ignore: Ignore this behaviour\n - error: Treat as error\n\n --show-version\n Show crate versions in the output\n\n --fix\n Try to automatically fix the problems\n\n --modify-paths <MODIFY_PATHS>\n \n\n --fix-dependency <FIX_DEPENDENCY>\n Fix only issues with this package as dependency\n\n --fix-package <FIX_PACKAGE>\n Fix only issues with this package as feature source\n\n -h, --help\n Print help (see a summary with '-h')\n"

0 comments on commit 99427c2

Please sign in to comment.