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 dist folder #2261

Closed
chassq opened this issue Mar 8, 2015 · 7 comments
Closed

Add dist folder #2261

chassq opened this issue Mar 8, 2015 · 7 comments

Comments

@chassq
Copy link

chassq commented Mar 8, 2015

Hey Guys,

Great lib! Would you mind adding a dist folder with compiled code? It would be nice to have when we use bower to install pkg. Right now we only seem to get the source and would have to compile ourselves. Would be a nice convenience!

Thanks!

@adunkman
Copy link
Contributor

adunkman commented Mar 9, 2015

@chassq check out the releases section for compiled code. We’re not so happy to include build artifacts in the source repository, since, well, they’re build artifacts.

Thanks for the interest, though!

@adunkman adunkman closed this as completed Mar 9, 2015
@chassq
Copy link
Author

chassq commented Mar 9, 2015

The issue is your releases section is not set up for a bower pull. You should consider it. Many other javascipt libraries do this and have a build directory on the main release branch.

Sent from my Windows Phone


From: Andrew Dunkmanmailto:notifications@github.com
Sent: ý3/ý9/ý2015 8:46 AM
To: harvesthq/chosenmailto:chosen@noreply.github.com
Cc: Charles Laymonmailto:charlesl@stepquest.com
Subject: Re: [chosen] Add dist folder (#2261)

@chassqhttps://github.com/chassq check out the releaseshttps://github.com/harvesthq/chosen/releases section for compiled code. We’re not so happy to include build artifacts in the source repository, since, well, they’re build artifacts.

Thanks for the interest, though!


Reply to this email directly or view it on GitHubhttps://github.com//issues/2261#issuecomment-77846545.

@tjschuck
Copy link
Member

tjschuck commented Mar 9, 2015

Bower does not currently support adding a post-install/build step. We don't provide compiled sources in the repos for many reasons which have been repeatedly stated throughout the issue tracker.

However, you can specify that you'd like to use the release ZIP, which includes the compiled and minified sources.

Either install from the command line with

$ bower install https://github.com/harvesthq/chosen/releases/download/v1.4.0/chosen_v1.4.0.zip

or add Chosen to your own project's bower.json file, like:

{
  "name": "my-project",
  "version": "1.0.0",
  "dependencies": {
    "jquery": "1.11.0",
    "chosen": "https://github.com/harvesthq/chosen/releases/download/v1.4.0/chosen_v1.4.0.zip"
  }
}

See #1509, #1472, and #1333 for more info.

This is even documented in the README.

@pfiller
Copy link
Contributor

pfiller commented Mar 28, 2015

bower install chosen

As of 1.4.2, Chosen supports bower installs. Read the 1.4.2 release notes for details.

@alvarotrigo
Copy link

alvarotrigo commented Sep 29, 2016

I personally always find the dist folder useful. Even to just compare the amount and size of code I'm going to include in my project or to just compare it with alternatives out there. In this case, select2 or selectize.

@stof
Copy link
Collaborator

stof commented Sep 29, 2016

@alvarotrigo having conflicts on all pull requests each time we merge one, because of the dist folder (and so being unable to merge 2 PRs in a row) is a huge burden for maintainers. If you want to see the code size, look at the other repository where we store built files (updated only for releases, not for every push into master)

@alvarotrigo
Copy link

@stof I see... Just as a though. Maybe pull request can be done in a dev branch with no dist folder in it ?

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

No branches or pull requests

6 participants