Skip to content

Bump json-schema/JSON-Schema-Test-Suite to 1.1.2 #264

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

Conversation

mirfilip
Copy link
Contributor

@mirfilip mirfilip commented May 6, 2016

/cc @bighappyface

No effort move, all tests still passing. I'm currently looking at using 1.2.0 (seen that)

@bighappyface
Copy link
Collaborator

@mirfilip please use a global ignore for the .idea directory. That is a user-specific ignore as not everyone uses the Jetbrains products.

Otherwise +1

@mirfilip
Copy link
Contributor Author

mirfilip commented May 9, 2016

@bighappyface I cannot use a global ignore as some projects I work with do keep this under version control. Anyway, it was rather a thing for every other contributor that uses Jetbrains stuff so that the IDE config doesn't get committed. I can always create an explicit repo exclude, so no problem.

I've checked some popular repos and they indeed refrain from specifying IDE specific stuff. Reverting that then.

@bighappyface
Copy link
Collaborator

@mirfilip thanks for the update. Last request: please squash this down.

@mirfilip mirfilip force-pushed the feature/upgrade-json-schema-test-suite-to-1.1.2 branch from 58bacc0 to a4d0b4c Compare May 9, 2016 21:19
@mirfilip
Copy link
Contributor Author

mirfilip commented May 9, 2016

@bighappyface Sure! Also, please see #265 as it supersedes this PR.

@bighappyface bighappyface merged commit a2be633 into jsonrainbow:master May 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants