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

Track PR #323 #372

Closed
28 tasks done
tobie opened this issue Jun 21, 2017 · 4 comments
Closed
28 tasks done

Track PR #323 #372

tobie opened this issue Jun 21, 2017 · 4 comments

Comments

@tobie
Copy link
Collaborator

tobie commented Jun 21, 2017

Following PR #323, file issues with:

@tobie
Copy link
Collaborator Author

tobie commented Jun 23, 2017

All issues filed; closing.

@tobie tobie closed this as completed Jun 23, 2017
@domenic
Copy link
Member

domenic commented Jun 23, 2017

What about the parsers? They should start rejecting serializer syntax, and accepting toJSON if they don't already.

@tobie
Copy link
Collaborator Author

tobie commented Jun 23, 2017

So the parsers already accept toJSON as it's just a regular operation (usage wasn't blocked at the grammar level). But you're right that they can get rid of lots of code now that serializers are removed. I hadn't thought about that. :)

@tobie tobie reopened this Jun 23, 2017
@tobie
Copy link
Collaborator Author

tobie commented Jun 23, 2017

Also filed bugs with vendors and test suites. Think I'm done this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants