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.

Sign up for GitHub

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

Jump to bottom

Fixes #244 - try parsing as JSON if filename is a named pipe/fifo #246

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Fixes #244 - try parsing as JSON if filename is a named pipe/fifo #246

wants to merge 7 commits into from

Conversation

DavidBiesack
Copy link

Also fixes issue #245

@DavidBiesack
resolve tsc compiler errors with err.message, err.code
5b80fdc
@DavidBiesack
resolve tsc compiler errors with err.message, err.code
63c0078
@DavidBiesack
Let file parsing work for named pipes (fifo)
031dee0 
Fixes issue ajv-validator#244

Default to parsing JSON if filename (a named fifo) is passed
that does not have a recognized extension - parse rather than
return `{}` for the schema, which accepts all input.

Added code to allow tsc to compile without error when accessing
`err.message` etc.
@DavidBiesack
increment package patch version to 5.0.1
beb2d09
@DavidBiesack
address eslint issues
69ce61e
@DavidBiesack
add test cases for file names with no extension - assume JSON
da0ccbf
@DavidBiesack
split out new tests with separate it functions
43b8139
@DavidBiesack
Copy link
Author

See also #242 which addresses #244 in a different way. I stumbled upon that after creating my PR. #242 was created a few months ago, not reviewed/merged...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@DavidBiesack