Skip to content

Conversation

@eguiraud
Copy link
Contributor

@eguiraud eguiraud commented Apr 5, 2017

clang-format will not be happy with the very long lines in the user guide, but on the other hand markdown only accepts table rows on a single line, so I think this case calls for an exception.

eguiraud added 2 commits April 5, 2017 12:31
Markdown tables seem to not like line breaks inside a cell.
Clang-format, on the other hand, did not like our very long lines,
breaking the docs.
@phsft-bot
Copy link

Starting build on centos7/gcc49, mac1011/native, slc6/gcc49, slc6/gcc62, ubuntu14/native and CMake flags -Dccache=ON -Dimt=OFF

@phsft-bot
Copy link

Starting build on centos7/gcc49, mac1011/native, slc6/gcc49, slc6/gcc62, ubuntu14/native and CMake flags -Dccache=ON -Dimt=OFF

@eguiraud
Copy link
Contributor Author

eguiraud commented Apr 5, 2017

travis complains about the long lines in the markdown tables as expected.
jenkins seems to have failed a build on mac1011 due to his own issues.

@peremato
Copy link
Contributor

peremato commented Apr 5, 2017

@bluehood why you run it with -Dimt=OFF?

@eguiraud
Copy link
Contributor Author

eguiraud commented Apr 5, 2017

@peremato it's not me, now the jenkins bot is more verbose and specifies the options and platforms it builds with. those are the defaults.

@dpiparo
Copy link
Member

dpiparo commented Apr 5, 2017

Hi @peremato, the mixture of the imt and non-imt builds was not changed. Before proceeding to make the non-imt build a special case we decided at the PPP meeting to wait to have all tests green.

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.

4 participants