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.

Sign up for GitHub

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

Jump to bottom

Throw an error if techOrder isn't in options. For #1998 #2097

Closed
wants to merge 3 commits into from
Closed

Throw an error if techOrder isn't in options. For #1998 #2097

wants to merge 3 commits into from

Commits on Apr 29, 2015

  1. Throw an error if techOrder isn't in options. For videojs#1998 

    Bail with a friendly error if techOrder isn't defined during player construction. Add in a qunit test page at the root of the tests directory for simpler browser debugging.
    @dmlap
    dmlap committed Apr 29, 2015
    Configuration menu
    Copy the full SHA
    45cc59d View commit details
    Browse the repository at this point in the history

  2. Don't fail the build for coveralls 

    Coveralls is behaving unreliably so treat a failure as a warning.
    @dmlap
    dmlap committed Apr 29, 2015
    Configuration menu
    Copy the full SHA
    8f77be1 View commit details
    Browse the repository at this point in the history

  3. Remove extra div. Add styles. 

    The test fixture is created in the globals shim now so remove the inline element. Add the default video.js styles to the page until we turn on the script-based injection.
    @dmlap
    dmlap committed Apr 29, 2015
    Configuration menu
    Copy the full SHA
    67bb555 View commit details
    Browse the repository at this point in the history