Skip to content

Update bower.json #72

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

Closed
wants to merge 1 commit into from
Closed

Update bower.json #72

wants to merge 1 commit into from

Conversation

wodka
Copy link

@wodka wodka commented Aug 11, 2014

add bower.json

@minodisk
Copy link

+1
main property is required for browserify + debowerify

@mgol
Copy link
Member

mgol commented Oct 19, 2014

@minodisk This should IMO just have proper fields set in package.json so that no debowerify is needed. cc @dmethvin

@wodka Version master looks weird; it should match what's in package.json, namely 2.1.3pre (though it should be changed in both places to 2.1.3-pre for the same reasons jQuery Core has changed that). We should also generate bower.json only during the release, the same as we plan in Core (ref jquery/jquery#1673). Unless this repo is handled 100% manually? cc @scottgonzalez

@scottgonzalez
Copy link
Member

It should be handled by jquery-release. It just hasn't been updated to use it yet since the last release was almost two years ago.

@JAAulde
Copy link

JAAulde commented Mar 12, 2015

bower.json is still missing the main property, and wiredeps needs it too.

@gnarf
Copy link
Member

gnarf commented Jan 14, 2016

I think we updated this with #87 - if not, new PR's accepted

@gnarf gnarf closed this Jan 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

6 participants