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

Separating repo? #176

Closed
drewhamlett opened this issue Feb 27, 2013 · 9 comments
Closed

Separating repo? #176

drewhamlett opened this issue Feb 27, 2013 · 9 comments

Comments

@drewhamlett
Copy link

I'd like to use js-beautify as git submodule but I hate to pull down php and python code too.

Considering this is JS beautify, would it make any sense to put the JS version in it's own repo? Then call other ones like js-beautify-php js-beautify-python. Basically I'd like to have the JS versions of the beautify(JS, HTML, CSS)

Just a thought.

@einars
Copy link
Contributor

einars commented Feb 27, 2013

I really care for and maintain only js and python versions, it looks to me that by separating the repos they quickly and easily would to run out of sync (now in case of problems/bugs/etc I fix them both simultaneously).

Maybe a separate js-only branch with a semi-automatic export of the lib, something like it's currently done with gh-pages push 💡

@drewhamlett
Copy link
Author

Semi-automatic export JS-only branch sounds great to me. Just thought I'd pass it by you. I'd hate to have people rely on me to manually keep up with changes in JS-beautify in the brackets-beautify project. Thanks.

@einars
Copy link
Contributor

einars commented Mar 13, 2013

adding a cross-reference to #181, where the need for tagged release packages was mentioned

@evocateur
Copy link
Contributor

If you depend on https://npmjs.org/package/js-beautify, you'll only download the JS (achieved through liberal use of .npmignore, which omits the Python and PHP from the npm artifact).

@bitwiseman
Copy link
Member

@drewhjava - It looks like @evocateur is keeping this version of the npm package up-to-date, and it only has the javascript in it. Will that be sufficient? Can we close this issue?

@drewhamlett
Copy link
Author

@bitwiseman @evocateur Ok cool. That looks like it will work. Thanks for the help!

@drewhamlett
Copy link
Author

@evocateur @bitwiseman Actually, now that look at it, the npm version doesn't have the html and css beautify files.

This one includes html and css but is not updated regularly.

https://npmjs.org/package/beautifier

I mean maybe it's not a huge deal. I just notice js-beautify gets updated regularly and I hate to leave users in the dark.

@bitwiseman
Copy link
Member

@drewhjava - Please open a new issue for this, either here or in @evocateur 's awesome fork https://github.com/evocateur/js-beautify , since he's running the show for npm packaging.

@evocateur
Copy link
Contributor

@drewhjava Sorry, I had Issues disabled in my fork. I've re-enabled them, and will be opening an issue shortly.

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

No branches or pull requests

4 participants