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

Add nix to default languages #2262

Merged
merged 1 commit into from Apr 12, 2024
Merged

Add nix to default languages #2262

merged 1 commit into from Apr 12, 2024

Conversation

Janik-Haag
Copy link
Contributor

closes #2261

While I could maintain my own highlight.js for my Project, this would probably benefit a few users. Esp. since mdBook is already quite common in the nix-community, for example the Nix Reference Manual uses it.

@rustbot rustbot added the S-waiting-on-review Status: waiting on a review label Dec 7, 2023
@deemp
Copy link

deemp commented Jan 13, 2024

@ehuss, @Dylan-DPC , please, review. I'd like to use Nix highlighting in numtide/devshell docs.

@Janik-Haag
Copy link
Contributor Author

You can relatively easily use it already, but native support would be nicer.
I'm already deploying it for nixos-dns, see here for the code https://github.com/Janik-Haag/NixOS-DNS/blob/main/docs/default.nix#L30

Copy link

@jalil-salame jalil-salame left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@ehuss ehuss added the A-Highlighting Area: Code colored syntax highlighting label Feb 25, 2024
@deemp
Copy link

deemp commented Apr 5, 2024

@ehuss, @jalil-salame, can you please merge?

@jalil-salame
Copy link

@deemp I just reviewed it, I don't have merge rights c:

@ehuss
Copy link
Contributor

ehuss commented Apr 10, 2024

Thanks for the PR! Unfortunately we are not able to update highlight.js. Can you make sure this is built with 10.1.1?

@ehuss ehuss added S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. and removed S-waiting-on-review Status: waiting on a review labels Apr 10, 2024
@Janik-Haag
Copy link
Contributor Author

Hey @ehuss I updated highlight js with the exact 10.1.1 version. Is there anything else I should do?

Copy link
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ehuss ehuss added this pull request to the merge queue Apr 12, 2024
Merged via the queue into rust-lang:master with commit c671c2e Apr 12, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Highlighting Area: Code colored syntax highlighting S-waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add nix-lang highlighting by default
5 participants