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.

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

Updated header and Viacom image link in 'They use hls.js in production!' section of README #2127

Merged
merged 1 commit into from
Feb 12, 2019

Conversation

brentsmith-dev
Copy link
Contributor

Viacom image should point at new domain and use https
Changing 'they use hls.js in production !' to 'They use hls.js in production!'

This PR will...

Update viacom image url and fix header text in 'they use hls.js in production !' section of README.md

Why is this Pull Request needed?

Clean up the header and ensure the image link doesn't break in the near future.

Are there any points in the code the reviewer needs to double check?

No

Resolves issues:

None

Checklist

  • [:white_check_mark:] changes have been done against master branch, and PR does not conflict
  • [:white_check_mark:] new unit / functional tests have been added (whenever applicable)
  • [:white_check_mark:] API or design changes are documented in API.md

…com image link to new domain and using https
@johnBartos
Copy link
Collaborator

@brentsmith-dev Thanks! Always appreciate grammatical corrections

@johnBartos johnBartos merged commit fd7f6d3 into video-dev:master Feb 12, 2019
dyue added a commit to dyue/hls.js that referenced this pull request Mar 9, 2019
* origin/master: (59 commits)
  Change networkDetails type to "unknown".
  mp4-remuxer / remuxAudio() : recompute mdatSize / dont rely on audioTrack.len (video-dev#2096)
  Typo fix for destory -> destroy.
  Change responseType to string
  Callbacks are defined, removing TODO.
  video-devGH-2082: Replacing usage of Array.prototype.find/findIndex (video-dev#2117)
  Reduce GC pressure from ID3._utf8ArrayToStr (video-dev#2035)
  (misc): converting binary search to typescript (video-dev#2129)
  (misc) Removing IE8-specific (dead) vtt/cue code
  Fixed case of 'They use hls.js in production!' header and updated Viacom image link to new domain and using https (video-dev#2127)
  remove events/EventEmitter && move Observer to ts (video-dev#2097)
  Basic typescript of event handler.
  Update webpack to use babel loader with support for TS. (video-dev#2119)
  Allow console statement in test bench and saucelabs code.
  Lint fixes
  Playlist Loader conversion to TypeScript.
  Playlist-loader rename.
  Change IV handling for initSegments to just log a warning.
  Add handling for AES-128 encrypted initialization segments needing IV.
  (misc) Adding ace editor to demo to enable advanced customization (video-dev#2077)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants