Skip to content
This repository has been archived by the owner on Dec 2, 2022. It is now read-only.

Added Jade.sublime-build #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added Jade.sublime-build #16

wants to merge 1 commit into from

Conversation

jansanchez
Copy link

Added Jade.sublime-build for Sublime Text users, additionally have support for linux and osx.

@jansanchez
Copy link
Author

would be good, to be implemented the sublime-build to be installed from the package manager sublime, thanks

@davidrios
Copy link
Owner

You have sent a pull request from a fork of another repository. Besides, I don't think this is a very interesting addition, definitely not with the "--watch" param.

@jansanchez
Copy link
Author

Sorry, it's true... the pull request takes the last fork that I made to "jade-tmbundle".

Just make the right change to make a new pull request as there are many people using jade with sublime-text, so I feel it would be of much use to install all in one, but that's only if you give me the approval to submit.

Thanks for reply, I wonder why it is not optimal or good to have the parameter "- watch"?

@davidrios
Copy link
Owner

I don't think this is interesting because of the way people generally (as far as my experience goes) deal with the compilation of jade files, which is in a more controlled manner and not randomly from the editor. The watch parameter is especially bad because it creates a persistent process.

@b3n01t
Copy link

b3n01t commented Jul 13, 2014

In combination with SublimeOnSaveBuild, it is great.
Here is the build system I use.
https://gist.github.com/b3n01t/5b6dbddfc77faba405bc#file-jade-sublime-build

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants