Skip to content

Commit

Permalink
fix(bower.json): JS files should not be ignored
Browse files Browse the repository at this point in the history
  • Loading branch information
eduardomb committed Jan 16, 2014
1 parent aa03555 commit ccdab19
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion bower.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
},
"ignore": [
"**/.*",
"*.js",
"node_modules",
"bower_components",
"component.json",
Expand Down

15 comments on commit ccdab19

@ericclemmons
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eduardomb Does this require a new tag @ v0.2.9? bower install --save angular-ui-router installs at #0.2.8, which resolves to: https://github.com/angular-ui/ui-router/archive/0.2.8.tar.gz

If you inspect this, you'll see that bower.json still has the line in place.

@eduardomb
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ericclemmons, there is a release for this fix: 0.2.8-bowratic-tedium. You should try:

bower install --save angular-ui-router#0.2.8-bowratic-tedium

@ericclemmons
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, that specificity resolved it. We were @ ~0.2.7, so we were quiet surprised when 0.2.8 still had the issue, but bowratic-tedium solved it. Looking forward to ~0.2.9 so we can get back to auto-updated bug fixes!

@teamon
Copy link

@teamon teamon commented on ccdab19 Jan 23, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please relese 0.2.8.1? This -bowratic-tedium suffix doesn't play well with other package manager systems like http://rails-assets.org

@stephenplusplus
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ivancamilov
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I concur with @teamon and @ericclemmons. -bowratic-tedium is not Semver compliant.

@sudodoki
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having "angular-ui-router": "~0.2.0", set in my bower.json breaks my build. Rolling back to 0.2.6 for time being.

@cgross
Copy link

@cgross cgross commented on ccdab19 Feb 6, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Every new user following the getting started instructions and does bower install angular-ui-router still has a broken package until a new release is made.

@nadeesha
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 for 0.2.8.1. What would be the rationale behind ignoring js files?

@stephenplusplus
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ProLoser
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think 0.2.8.1 is valid semver

@nateabele your 0.2.8-whatever didn't work because according to semver it's considered lower than 0.2.8, so it should have been 0.2.9-whatever

@nateabele
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0.2.9 will be released this weekend. Bower people, please kindly cool your jets. 🍻

@ericclemmons
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update @nateabele! <3 this project! (It made our internal application a breeze to work with :D)

@kevinrenskers
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any news on 0.2.9?

@IlanFrumer
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.