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 dsonlint link #25

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Add dsonlint link #25

wants to merge 8 commits into from

Conversation

AnEmortalKid
Copy link
Member

Fixes #24

Add dsonlint link

@AnEmortalKid AnEmortalKid changed the title Update index.html Add dsonlint link Sep 15, 2019
@vpzomtrrfrt
Copy link
Member

Looks like this is effectively a frontend for https://github.com/JDeuce/dson_parser, should we put it inline with that?

@JDeuce
Copy link

JDeuce commented Sep 21, 2019

That might be reasonable. It's a standalone tool that can help show off the spec a little more than the library. I expect nobody to ever care about dson_parser but maybe someone will use dsonlint if they come across the spec. So you could just replace the one with the other if concerned about too many links to my stuff.

Also updated the link to the spec on dsonlint from my local mirror to dogeon.xyz, since I made dsonlint in that limbo period where no one was hosting the spec.

@AnEmortalKid
Copy link
Member Author

should we put it inline with that?

Inlined

@@ -198,7 +198,7 @@ <h1 align="center">wow. such Doge Serialized Object Notation.</h1>
<hr>
<div style="float: left">
<p>such DSON. very parse. so open source:</p>
<ul>
<ul>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like this space shouldn't be here?

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.

dsonlint
3 participants