Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Commit

Permalink
Remove contributing documentation about bumping LibSass
Browse files Browse the repository at this point in the history 
I can't imagine accepting a PR with a LibSass bump.

It'd be painful to audit. LibSass should be handled by core.
  • Loading branch information
@xzyfer
xzyfer committed Jan 7, 2017
1 parent 2d6d19e commit ae31ada
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions CONTRIBUTING.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -42,11 +42,6 @@ If the bug fix requires modifying any of the C++ files in the src/ directory, yo

Alternatively, the `scripts/build.js` build script has several different command line flags that can be passed by running: `node scripts/build.js -[flag]`

If the bug fix requires updating the version of libSass, you'll need to update its git submodule.

1. Move into node-sass's libSass directory with: `cd src/libsass`
2. Look for a new version of libSass with: `git tag` and check it out with: `git checkout [tag]`
3. Then return to root of the node-sass repository and add the change to your feature branch.

## Reporting Sass compilation and syntax issues

Expand Down

0 comments on commit ae31ada

Please sign in to comment.