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

Modernize diet related examples #551

Merged
merged 8 commits into from Feb 26, 2014

Conversation

Projects
None yet
2 participants
@nazriel
Contributor

nazriel commented Feb 25, 2014

Using doctype html instead of !!! 5 and few more tweaks.

Rest of examples will be updated soon aswell.

In process of updating I also found a bug in vibe.http.form.
See #Workaround to make form_interface example compile commit

@s-ludwig s-ludwig referenced this pull request Feb 26, 2014

Merged

Add examples to Travis #552

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Feb 26, 2014

Member

Sorry for causing conflicts - the two compile fixes for the form interface (example) are already merged with #552. Can you skip/rebase those?

Member

s-ludwig commented Feb 26, 2014

Sorry for causing conflicts - the two compile fixes for the form interface (example) are already merged with #552. Can you skip/rebase those?

@nazriel

This comment has been minimized.

Show comment
Hide comment
@nazriel

nazriel Feb 26, 2014

Contributor

@s-ludwig no problem at all.

Should be good now?

Contributor

nazriel commented Feb 26, 2014

@s-ludwig no problem at all.

Should be good now?

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Feb 26, 2014

Member

Thanks! Yes, looks good and even passes the new Travis script as expected ;)

Member

s-ludwig commented Feb 26, 2014

Thanks! Yes, looks good and even passes the new Travis script as expected ;)

s-ludwig added a commit that referenced this pull request Feb 26, 2014

@s-ludwig s-ludwig merged commit e1f199b into vibe-d:master Feb 26, 2014

1 check passed

default The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment