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

[trivial] Enable Markdown table syntax #471

Merged
merged 1 commit into from
Nov 16, 2016

Conversation

wilzbach
Copy link
Member

Simply enables the use of the markdown table syntax with the Vibe.d parser

Introduced in vibe-d/vibe.d@5480a5c

@stonemaster
Copy link
Collaborator

I'd like to wait until 0.7.30 final before merging this. We rely on vibe for the whole web service so I don't feel comfortable with an alpha release..

@wilzbach wilzbach changed the title [trivial] Enable Markdown table syntax [blocked][trivial] Enable Markdown table syntax Aug 25, 2016
@stonemaster
Copy link
Collaborator

Vibe.d is already in rc.2 so we're getting closer to this!

@wilzbach wilzbach changed the title [blocked][trivial] Enable Markdown table syntax [trivial] Enable Markdown table syntax Nov 3, 2016
@wilzbach wilzbach force-pushed the use-markdown-tables branch 3 times, most recently from 9416d36 to 050aa00 Compare November 3, 2016 14:42
@wilzbach
Copy link
Member Author

wilzbach commented Nov 3, 2016

Vibe.d is already in rc.2 so we're getting closer to this!

It was released :)
Updated this PR & while I was at it also the copyright year.

@PetarKirov
Copy link
Member

Since dmd is now at 2.072 on Travis our build fails because one of our DUB dependencies - dyaml - does not compile, because of the more rigorous @safe-ty checks that were introduced with the following DMD PRs: dlang/dmd/pull/5852, dlang/dmd#5940, dlang/dmd#5876 and dlang/dmd#5860.

@PetarKirov
Copy link
Member

I'll prepare a PR for dyaml asap.

@PetarKirov
Copy link
Member

Done: dlang-community/D-YAML/pull/49.

@stonemaster
Copy link
Collaborator

@ZombineDev thanks for the effort with Dyaml!

@PetarKirov
Copy link
Member

No problem, I hope to have more spare time in the coming weeks to work on dlang-tour :)

@stonemaster
Copy link
Collaborator

Ditto. I am totally busy in real life but I hope to have finally more spare time in the next year :-)

@PetarKirov PetarKirov merged commit baa51fe into dlang-tour:master Nov 16, 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.

None yet

3 participants