-
Notifications
You must be signed in to change notification settings - Fork 258
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
docs(scully): describe syntax highlighting #120
docs(scully): describe syntax highlighting #120
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@Villanuevand is this the correct place for this? If not, can you work with @d-koppenhagen to resolve this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I not sure if "Add Syntax Highlighting" should be part of the main menu.
I think we need to add this section like "enhancement" ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@d-koppenhagen
We are ready to merge after add link to utils.md
Closes #111
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Syntax Highlighting is not described currently
Issue Number: #111
What is the new behavior?
Syntax Highlighting is described after
Does this PR introduce a breaking change?
Other information