Skip to content

Syntax highlighting as console #523

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

Merged
merged 1 commit into from
Jul 1, 2022
Merged

Syntax highlighting as console #523

merged 1 commit into from
Jul 1, 2022

Conversation

toastal
Copy link
Contributor

@toastal toastal commented Jun 29, 2022

The code blocks are shell sessions and not shell scripts which is why the highlighted oddly (install a different color)


Out of scope project suggestions:

  • Support a mirror on an open source Git forge
  • Support an open source communication alternatives like IRC or Matrix

The code blocks are shell sessions and not shell scripts which is why the highlighted oddly (`install` a different color)

---

Out of scope project suggestions:

* Support a mirror on an open source Git forge
* Support an open source communication alternatives like IRC or Matrix
@fdncred
Copy link
Contributor

fdncred commented Jun 29, 2022

Same as the nushell issue, I'm not sure how I feel about tables in blue. I'm not really sure what console buys us. I'd rather just have our own custom theme somehow.

@fdncred
Copy link
Contributor

fdncred commented Jul 1, 2022

I think we'll keep it how it is now. Thanks for your time in this submission and trying to make us look better.

@fdncred fdncred closed this Jul 1, 2022
@toastal
Copy link
Contributor Author

toastal commented Jul 1, 2022

This one doesn't have a table. These shell session examples are both technically the wrong syntaxes.

@fdncred fdncred reopened this Jul 1, 2022
@fdncred
Copy link
Contributor

fdncred commented Jul 1, 2022

ok

@fdncred fdncred merged commit e9f3f28 into nushell:main Jul 1, 2022
ayax79 pushed a commit to ayax79/nushell.github.io that referenced this pull request Jun 26, 2024
Co-authored-by: agent <agent@nuc>
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.

2 participants