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

Remove lax schema variants #31

Open
mvahowe opened this issue Feb 6, 2018 · 1 comment
Open

Remove lax schema variants #31

mvahowe opened this issue Feb 6, 2018 · 1 comment
Milestone
Sprint 9

Comments

@mvahowe
Copy link
Contributor

mvahowe commented Feb 6, 2018

We added this for 2.0 migration, where the lax variant of the schema more or less ignores the content of elements. At this point I believe that we have decided this was not as good an idea as we initially thought, and that we are no longer using the lax variant anywhere. Maintaining it is hassle (I believe it is broken in my latest PR) so can we remove it? @smorrison @ericpyle
@mvahowe mvahowe added this to the Sprint 9 milestone Feb 6, 2018
@mvahowe mvahowe self-assigned this Feb 6, 2018
@smorrison
Copy link
Contributor

We can remove it. It's only used in dead code, which should also be removed.

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

No branches or pull requests
2 participants
@smorrison @mvahowe