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

fix #973 #1801

Merged
merged 8 commits into from Jun 9, 2015
Merged

fix #973 #1801

merged 8 commits into from Jun 9, 2015

Conversation

@ralsina
Copy link
Member

ralsina commented Jun 9, 2015

Sadly can't do it in all the compiler plugins in the plugins repo because tht would break them for earlier versions.

@Kwpolska
Copy link
Member

Kwpolska commented Jun 9, 2015

don’t we do it in all the other compilers? What about reST?

@ralsina
Copy link
Member Author

ralsina commented Jun 9, 2015

@Kwpolska we don't do it for pandoc (and maybe we should!), added it for rest.

@Kwpolska
Copy link
Member

Kwpolska commented Jun 9, 2015

We might try doing it for Pandoc and PHP.

@ralsina
Copy link
Member Author

ralsina commented Jun 9, 2015

Main problem with pandoc is that we have no idea what's inside the file.

@Kwpolska
Copy link
Member

Kwpolska commented Jun 9, 2015

Merge when Travis is done testing your branch then.

ralsina and others added 4 commits Jun 9, 2015
doh
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Kwpolska added a commit that referenced this pull request Jun 9, 2015
@Kwpolska Kwpolska merged commit f429714 into master Jun 9, 2015
3 checks passed
3 checks passed
continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@Kwpolska Kwpolska deleted the improve-973 branch Jun 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.