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

docs: remove flattr-badge.png image #1395

Merged
merged 1 commit into from
Dec 29, 2017

Conversation

amurzeau
Copy link
Contributor

It was used in README.rst before, but is no longer used since commit
b4470f3.

Also remove _static directory and its reference from docs/conf.py as it
only contains that image.

It was used in README.rst before, but is no longer used since commit
b4470f3.

Also remove _static directory and its reference from docs/conf.py as it
only contains that image.
@codecov
Copy link

codecov bot commented Dec 29, 2017

Codecov Report

Merging #1395 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1395   +/-   ##
=======================================
  Coverage   46.88%   46.88%           
=======================================
  Files         104      104           
  Lines        7421     7421           
=======================================
  Hits         3479     3479           
  Misses       3942     3942

@gravyboat
Copy link
Member

Good catch thanks @amurzeau!

@gravyboat gravyboat merged commit ccb5850 into streamlink:master Dec 29, 2017
@amurzeau
Copy link
Contributor Author

Don't thank me, it was found by Paul Wise when reviewing the package :)
That image had an unclear license and so was to be removed from source .tar.gz before entering Debian.

Have a nice day

@amurzeau amurzeau deleted the remove-flattr-badge branch January 26, 2018 23:21
mkbloke pushed a commit to mkbloke/streamlink that referenced this pull request Aug 18, 2020
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.

None yet

2 participants