-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Fork json-comments-rs and remove trailing commas #9032
Conversation
72e6605
to
931021e
Compare
028b716
to
da00883
Compare
@@ -0,0 +1,546 @@ | |||
//! `json_comments` is a library to strip out comments from JSON-like test. By processing text |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
new top-level folder? or should we put it in packages? should we move all the crates out of packages?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The idea being that stuffing all of the crates into packages/utils
is getting more and more confusing, so I would suggest putting all of the somewhat general purpose crates into crates
itself (so dev-dep-resolver and node-resolver-rs). But you wanted to move them out of the monorepo anyway 🤷
Otherwise I can put this into packages/utils
as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that makes sense.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can pull out the unused stuff later.
Closes #9029
https://github.com/tmccombs/json-comments-rs plus