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

fix: Properly display unmount as a command #16041

Merged
merged 5 commits into from
Apr 21, 2021
Merged

Conversation

agg23
Copy link
Contributor

@agg23 agg23 commented Apr 16, 2021

User facing changelog

Entry for unmount in the CommandLog is more prominent and displays more information.

How has the user experience changed?

Before:
Screen Shot 2021-04-16 at 10 40 39 AM

After:
Screen Shot 2021-04-16 at 11 17 26 AM

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Apr 16, 2021

Thanks for taking the time to open a PR!

lmiller1990
lmiller1990 previously approved these changes Apr 19, 2021
Copy link
Contributor

@lmiller1990 lmiller1990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, definitely an improvement.

@cypress
Copy link

cypress bot commented Apr 19, 2021



Test summary

8043 0 101 3Flakiness 0


Run details

Project cypress
Status Passed
Commit a9ef456
Started Apr 21, 2021 12:55 PM
Ended Apr 21, 2021 1:06 PM
Duration 11:07 💡
OS Linux Debian - 10.8
Browser Multiple

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@agg23 agg23 changed the base branch from agg23/ReactRerender to develop April 20, 2021 15:35
@agg23 agg23 dismissed lmiller1990’s stale review April 20, 2021 15:35

The base branch was changed.

@agg23 agg23 requested a review from a team as a code owner April 20, 2021 15:35
@agg23 agg23 requested review from chrisbreiding and kuceb and removed request for a team April 20, 2021 15:35
@lmiller1990 lmiller1990 self-requested a review April 21, 2021 01:36
@lmiller1990
Copy link
Contributor

lmiller1990 commented Apr 21, 2021

Should this be targeting master so it gets released straight away? I think there is a bot that backmerges master in to develop.

Either way added a ✅ , merge when you are ready.

@agg23 agg23 merged commit 4002e4c into develop Apr 21, 2021
tgriesser added a commit that referenced this pull request Apr 21, 2021
…press into tgriesser/chore/improve-ci

* 'tgriesser/chore/improve-ci' of github.com:cypress-io/cypress:
  fix(deps): update dependency systeminformation to version 5.6.4 🌟 (#15819)
  docs: fix a typo of package name [skip ci] (#15783)
  chore: deduplicate yarn.lock (#15988)
  fix(webpack): allow load custom asset on windows (#16099)
  fix: Properly display unmount as a command (#16041)
  fix(component-testing): correct imports for relative paths in cypress.json  (#16056)
  Fixed missing SearchInput and improved SpecList scrolling (#16090)
  docs: update react docs (#16055)
  chore: Design System Cleanup (#16077)
  feat(component-testing): breaking: Add React rerender functionality (#16038)
  fix(component-testing): Increased timeout to allow  useEffect to trigger (#16091)
  chore: release @cypress/webpack-batteries-included-preprocessor-v2.2.1
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.

None yet

3 participants