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

Add support for 'data-raw' #14

Closed
wants to merge 12 commits into from
Closed

Add support for 'data-raw' #14

wants to merge 12 commits into from

Conversation

dnkmdg
Copy link
Contributor

@dnkmdg dnkmdg commented Oct 13, 2022

Added support for providing data-raw according to #8.

My reasoning for the proposed solution is that they should never be present simultaneously, and if data-raw isn't present, then go for default behavior with the -d|data-option, giving data-raw a slight precedence over data.

@dnkmdg
Copy link
Contributor Author

dnkmdg commented Oct 13, 2022

Fixtures post-json-data-raw available for testing

@jasonmccreary
Copy link
Collaborator

Sorry, this commit history is very odd. I would expect to just see one commit. Not everything from the other PR.

@dnkmdg
Copy link
Contributor Author

dnkmdg commented Oct 13, 2022

Yeah, I honestly haven't got the foggiest as to why it turned out that way. New PR with clean commits is up.

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.

None yet

2 participants