Skip to content
This repository was archived by the owner on Jul 23, 2025. It is now read-only.

Load all YAMLs in a directory if a path to the directory is given as ruleset-path #78

Merged
merged 1 commit into from
Sep 23, 2021

Conversation

lmt-swallow
Copy link
Member

Description

This PR changes the way to load rulesets following ruleset-path cli opt. Now users can use multiple rulesets with a single command as follows:

shisho check <ruleset-dir> <src-dir>

Checklist

  • I opened a draft PR or added the [WIP] to the title if my PR is not ready for review.
  • I have reviewed the code by myself.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have added tests enough to show how your code behaves and that your code works as expected.

Additional Notes

N/A

… ruleset-path

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>
@netlify
Copy link

netlify bot commented Sep 23, 2021

✔️ Deploy Preview for cranky-fermi-e6c846 canceled.

🔨 Explore the source changes: c4d4659

🔍 Inspect the deploy log: https://app.netlify.com/sites/cranky-fermi-e6c846/deploys/614bf765f70ba1000706e9a9

@lmt-swallow lmt-swallow merged commit 31d754e into main Sep 23, 2021
@lmt-swallow lmt-swallow deleted the feature/multiple-rule-files branch September 23, 2021 03:41
@lmt-swallow lmt-swallow mentioned this pull request Sep 23, 2021
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant