Skip to content
This repository has been archived by the owner on Jan 12, 2019. It is now read-only.

Repo Updates towards Beta #10

Merged
merged 1 commit into from
Feb 23, 2014
Merged

Repo Updates towards Beta #10

merged 1 commit into from
Feb 23, 2014

Conversation

tomjohnson916
Copy link
Contributor

  • add videojs build postinstall hook
  • update package references
  • add repository reference in package.json

- add videojs build postinstall hook
- update package references
- add repository reference in package.json
"grunt": "~0.4.1"
},
"dependencies": {
"video.js": "git+https://github.com/dmlap/video-js.git#v4.3.0-10",
"video.js": "git+https://github.com/videojs/video.js.git",
"videojs-swf": "git+https://github.com/seniorflexdeveloper/video-js-swf.git",
Copy link
Member

Choose a reason for hiding this comment

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

How long until the SWF changes can be merged into mainline? I'd prefer to hold off on updating dependencies until we can be on the official branch of everything.

Copy link
Member

Choose a reason for hiding this comment

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

Is videojs/video-js-swf#80 part of that?

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, that should be it.

dmlap added a commit that referenced this pull request Feb 23, 2014
@dmlap dmlap merged commit acbf792 into videojs:master Feb 23, 2014
@tomjohnson916
Copy link
Contributor Author

@dmlap - I talked to @heff on Friday and found out the tag versions of VJS now provide a full dist structure, removing the need for the postinstall hook moving forward. Will remove and set package.json dependency to a tagged version.

@dmlap
Copy link
Member

dmlap commented Feb 23, 2014

Cool, looking forward to that PR.

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

Successfully merging this pull request may close these issues.

3 participants