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

Introduce globbing for the JSONS input #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kiriakos
Copy link

Implemented globbing resolution based on @actions/glob [1].
Added test to validate globs are being read.
Fixes #19.

1: https://github.com/actions/toolkit/tree/main/packages/glob

Implemented globbing resolution based on @actions/glob [1].
Added test to validate globs are being read.
Fixes OrRosenblatt#19.

1: https://github.com/actions/toolkit/tree/main/packages/glob
@kiriakos
Copy link
Author

Okay, I see some issues with this. Using it as a github action like so:

      - name: Validate JSON
        uses: kiriakos/validate-json-action@e97bf4885314574701657ab73e5b7fceb3e07130
        with:
          schema: .github/schemas/donations-schema.json
        env:
          INPUT_SCHEMA: .github/schemas/donations-schema.json
          INPUT_JSONS: "/home/runner/work/Donations/Donations/*/donations.json"
          #OR
          INPUT_JSONS: "/*/donations.json"
          #OR
          INPUT_JSONS: "*/donations.json"
          #etc..

Does not work because the action adds the cwd to the paths at some point.... Why does it do that anyway?

Hans5958 pushed a commit to Hans5958/validate-json-action that referenced this pull request Jan 18, 2023
…and_yarn/types/node-18.7.18

Bump @types/node from 18.7.16 to 18.7.18
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.

Support glob for input paths
1 participant