Skip to content

Clean READMEs#3939

Merged
ndelangen merged 5 commits into
masterfrom
clean-readmes
Aug 1, 2018
Merged

Clean READMEs#3939
ndelangen merged 5 commits into
masterfrom
clean-readmes

Conversation

@igor-dv
Copy link
Copy Markdown
Member

@igor-dv igor-dv commented Aug 1, 2018

Issue: There is a few copy-pasted content from place to place, that is being hard to maintain because there is no single source of truth

What I did

  1. Removed badges from the inner packages (there was even broken greenkeeper badge somewhere)
  2. Commented out the BrowserStack since it is not implemented.
  3. Removed unused ancient images
  4. Moved all the copy-pasted demo.gifs to one place. If any framework want its own (like in marko), then it might be changed. I would even prefer some abstract Storybook drawings, because UI changes frequently

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 1, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3939   +/-   ##
=======================================
  Coverage   39.64%   39.64%           
=======================================
  Files         431      431           
  Lines        5449     5449           
  Branches      740      740           
=======================================
  Hits         2160     2160           
  Misses       2906     2906           
  Partials      383      383

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b147730...d86d77b. Read the comment docs.

@ndelangen
Copy link
Copy Markdown
Member

Nice cleanup!

@ndelangen ndelangen merged commit f373cf7 into master Aug 1, 2018
@ndelangen ndelangen deleted the clean-readmes branch August 1, 2018 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation maintenance User-facing maintenance tasks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants