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

report: add syntax highlighting #166

Merged
merged 1 commit into from
Sep 26, 2019

Conversation

piotr-binkowski
Copy link
Contributor

This fixes #162 and adds syntax highlighting using pygments.
Syntax highlighted sources look like this:
Screenshot_20190926_085418

@piotr-binkowski piotr-binkowski added bug Something isn't working enhancement New feature or request report labels Sep 26, 2019
@piotr-binkowski piotr-binkowski requested review from tgorochowik and a user September 26, 2019 07:04
@mithro
Copy link
Collaborator

mithro commented Sep 26, 2019

LGTM.

Makefile Outdated Show resolved Hide resolved
Signed-off-by: Piotr Binkowski <pbinkowski@antmicro.com>
@tgorochowik tgorochowik merged commit 99cb10a into chipsalliance:master Sep 26, 2019
@hzeller
Copy link
Collaborator

hzeller commented Sep 26, 2019

Very neat.

Can we modify the output a little with the configuration of pygments ?

  • on the top, there is some whitespace. The generated code seems to create some empty<h2></h2> for instance
  • If all line-numbers could contain an <a name="L42">42</a>, so that we can directly link from a logfile path/to/test/file.sv:42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request report
Projects
None yet
Development

Successfully merging this pull request may close these issues.

github.io report hosting: clicking on *.sv file in report downloads them vs. loading them in frame
4 participants