Skip to content

Commit

Permalink
build: bower.json: remove moot version field
Browse files Browse the repository at this point in the history
Per bower/spec@a325da3

closes #2144

bower.json: main: fix path to video.js

bower.json: main: video-js.css => video-js.scss

Per bower/spec#43 :
> Use source files with module exports and imports over pre-built distribution files.

Also, the example in that PR includes
/sass/motion.scss in and excludes
/dist/movement.css & /dist/movement.min.css
from its `main`.

bower.json: Set moduleType & use non-dist video.js

Again, see bower/spec#43
  • Loading branch information
cvrebert authored and heff committed May 14, 2015
1 parent 349ff89 commit 2bf0be7
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
6 changes: 3 additions & 3 deletions bower.json
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
{
"name": "video.js",
"description": "An HTML5 and Flash video player with a common API and skin for both.",
"version": "5.0.0-15",
"main": [
"dist/video-js/video.js",
"dist/video-js/video-js.css"
"src/js/video.js",
"src/css/video-js.scss"
],
"moduleType": "es6",
"keywords": [
"videojs",
"html5",
Expand Down
8 changes: 4 additions & 4 deletions grunt.js
Original file line number Diff line number Diff line change
Expand Up @@ -226,25 +226,25 @@ module.exports = function(grunt) {
options: {
release: 'major'
},
src: ['package.json', 'bower.json', 'component.json']
src: ['package.json', 'component.json']
},
minor: {
options: {
release: 'minor'
},
src: ['package.json', 'bower.json', 'component.json']
src: ['package.json', 'component.json']
},
patch: {
options: {
release: 'patch'
},
src: ['package.json', 'bower.json', 'component.json']
src: ['package.json', 'component.json']
},
prerelease: {
options: {
release: 'prerelease'
},
src: ['package.json', 'bower.json', 'component.json']
src: ['package.json', 'component.json']
},
css: {
options: {
Expand Down

1 comment on commit 2bf0be7

@wmcmurray
Copy link

Choose a reason for hiding this comment

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

What is the reason behind the removal of the version property ? It seems strange to me...

Please sign in to comment.