Skip to content

Conversation

@kgerheiser
Copy link

This fixes pre-processor highlighting

@krvajal
Copy link
Collaborator

krvajal commented Apr 12, 2018

Can you provide an image of the visual difference?

@krvajal krvajal mentioned this pull request Apr 12, 2018
4 tasks
@pedro-ricardo
Copy link
Collaborator

@krvajal, this is the preview you asked about this PR

Before:
bf

After:
af

@krvajal
Copy link
Collaborator

krvajal commented Apr 12, 2018

What about OpenMP directives. Are they highlighted properly?

@pedro-ricardo
Copy link
Collaborator

Nope ... issue #17 isn't corrected here.

It still looks like a comment
bf

Other editors like Kate/Vim/emacs color those as a comments too, so I don't have any reference to make that rule. I'll ask in the issue for a reference.

@krvajal krvajal changed the base branch from master to release/1.1.0 April 15, 2018 17:43
@codecov
Copy link

codecov bot commented Apr 15, 2018

Codecov Report

Merging #46 into release/1.1.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##           release/1.1.0      #46   +/-   ##
==============================================
  Coverage          13.21%   13.21%           
==============================================
  Files                  9        9           
  Lines                348      348           
  Branches              49       49           
==============================================
  Hits                  46       46           
  Misses               302      302

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 69fe9bf...68f393e. Read the comment docs.

@krvajal krvajal merged commit d59ad4c into fortran-lang:release/1.1.0 Apr 16, 2018
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.

3 participants