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

clear console on reload #638

Merged
merged 2 commits into from Oct 14, 2017
Merged

clear console on reload #638

merged 2 commits into from Oct 14, 2017

Conversation

xjlim
Copy link
Contributor

@xjlim xjlim commented Oct 13, 2017

fix #519

@codecov-io
Copy link

codecov-io commented Oct 13, 2017

Codecov Report

Merging #638 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #638      +/-   ##
==========================================
+ Coverage   96.03%   96.04%   +<.01%     
==========================================
  Files         101      101              
  Lines        1338     1339       +1     
  Branches      271      271              
==========================================
+ Hits         1285     1286       +1     
  Misses         51       51              
  Partials        2        2
Impacted Files Coverage Δ
src/rsg-components/Preview/Preview.js 85.71% <100%> (+0.25%) ⬆️

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 6dc568e...7f2863b. Read the comment docs.

@sapegin sapegin merged commit baf2cae into styleguidist:master Oct 14, 2017
@sapegin
Copy link
Member

sapegin commented Oct 14, 2017

Thanks, merged!

tizmagik added a commit that referenced this pull request Oct 24, 2017
* master:
  Refactor: Replace Markdown links with the Link component instead of the styles (#650)
  Docs: Clarify logging in Node API (#637)
  Docs: Correct the `static` modifier note (#643)
  Docs: Explain behavior for code blocks w/o language tag (#642)
  Docs: Fix invalid prop type warning (#639)
  Fix: Clear console on example reload (#638)
  Docs: Add credits for logo designers (#636)
  Fix: Update react-docgen (#635)
  Fix: Correctly print non-Styleguidist exception messages
  Fix: Fix validation error for uglifyjs-webpack-plugin
  Docs: Remove react-modal dependency, tweak examples
  Docs: Update react-styleguidist in CRA example (#632)

# Conflicts:
#	examples/basic/package-lock.json
#	examples/basic/package.json
#	examples/cra/package-lock.json
#	examples/customised/package-lock.json
#	examples/customised/package.json
#	examples/express/package-lock.json
#	examples/express/package.json
#	examples/sections/package-lock.json
#	examples/sections/package.json
#	examples/webpack/package-lock.json
#	examples/webpack/package.json
#	package-lock.json
#	package.json
@BruceHubbard
Copy link

What was the logic behind clearing the console? I recently updated my styleguidist version and I've noticed all of my React warnings are now hidden because the console gets cleared here.

@sapegin
Copy link
Member

sapegin commented Dec 2, 2017

@BruceHubbard I guess it shouldn’t be cleared on hard page reload, only on HMR updates. If it’s not the case, then it’s a bug.

@BruceHubbard
Copy link

@sapegin I'll open a new issue and see if others are having the same problem. On a hard page reload in Chrome I end up with zero React warnings when I know I should have some.

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

Successfully merging this pull request may close these issues.

Run console.clear() when components are reloaded
4 participants