Skip to content

languages/scss: WIP#1393

Draft
snoweuph wants to merge 1 commit intoNotAShelf:mainfrom
snoweuph:feature/scss
Draft

languages/scss: WIP#1393
snoweuph wants to merge 1 commit intoNotAShelf:mainfrom
snoweuph:feature/scss

Conversation

@snoweuph
Copy link
Copy Markdown
Collaborator

@snoweuph snoweuph commented Feb 4, 2026

Splits the SCSS/SASS from CSS module.
This allows for a different default LSP and extra tools.

WIP, waiting for: NixOS/nixpkgs#459753

Sanity Checking

  • I have updated the changelog as per my changes
  • I have tested, and self-reviewed my code
  • My changes fit guidelines found in hacking nvf
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt)
    • My code conforms to the editorconfig configuration of the project
    • My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • My code includes comments in particularly complex areas
    • I have added a section in the manual
    • (For breaking changes) I have included a migration guide
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html (manual, must build)
    • .#docs-linkcheck (optional, please build if adding links)
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.


cfg = config.vim.languages.scss;

defaultServer = ["scss-cssls"];
Copy link
Copy Markdown
Collaborator Author

@snoweuph snoweuph Feb 4, 2026

Choose a reason for hiding this comment

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

As soon as NixOS/nixpkgs#459753 is in nixos, I will add some-sass and make it default, as it's so much better for SCSS and also supports sass, unlike cssls.


defaultServer = ["scss-cssls"];
servers = {
scss-cssls = {
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

without prefix, this conflicts with the css module, as an lsp can only be registered once with its name

github-actions bot pushed a commit that referenced this pull request Feb 4, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 4, 2026

🚀 Live preview deployed from bfb6e48

View it here:

Debug Information

Triggered by: snoweuph

HEAD at: feature/scss

Reruns: 2415

@snoweuph
Copy link
Copy Markdown
Collaborator Author

snoweuph commented Feb 4, 2026

rebased

@snoweuph
Copy link
Copy Markdown
Collaborator Author

rebased -> showing I didnt forgot this and for my future sanity :D

@snoweuph
Copy link
Copy Markdown
Collaborator Author

rebased -> still waiting for nixpkgs

@snoweuph
Copy link
Copy Markdown
Collaborator Author

rebased

@snoweuph
Copy link
Copy Markdown
Collaborator Author

rebased for my sanity. still waiti9ng for upstream issue in nixpkgs

@snoweuph snoweuph force-pushed the feature/scss branch 2 times, most recently from 7989b05 to 87195ad Compare March 21, 2026 22:39
github-actions bot pushed a commit that referenced this pull request Mar 21, 2026
github-actions bot pushed a commit that referenced this pull request Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant