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

chore: remove Firefox < 80 auto garbage collection #17368

Merged
merged 3 commits into from
Jul 19, 2021

Conversation

flotwig
Copy link
Contributor

@flotwig flotwig commented Jul 16, 2021

This was a workaround added to fix a bug in FF < 80. Since Cy removed support for FF < 86 in #17355, this is effectively dead code.

This does not remove Cypress.backend('firefox:force:gc').

  • Closes

User facing changelog

n/a

Additional details

How has the user experience changed?

PR Tasks

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Jul 16, 2021

Thanks for taking the time to open a PR!

@flotwig flotwig changed the title chore: remove Firefox < 80 auto garbage collection fix: remove Firefox forced garbage collection Jul 16, 2021
@flotwig flotwig changed the title fix: remove Firefox forced garbage collection chore: remove Firefox < 80 auto garbage collection Jul 16, 2021
@flotwig flotwig force-pushed the remove-ff-auto-gc branch from 035aa47 to ad35833 Compare July 16, 2021 16:03
@cypress
Copy link

cypress bot commented Jul 16, 2021



Test summary

4122 0 50 2Flakiness 2


Run details

Project cypress
Status Passed
Commit 552598e
Started Jul 19, 2021 1:59 PM
Ended Jul 19, 2021 2:10 PM
Duration 10:42 💡
OS Linux Debian - 10.9
Browser Electron 91

View run in Cypress Dashboard ➡️


Flakiness

cypress/integration/commands/net_stubbing_spec.ts Flakiness
1 network stubbing > intercepting response > does not overwrite the json-related content-type header
2 network stubbing > waiting and aliasing > can timeout waiting on a single request using "alias.request"

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

@flotwig flotwig requested review from chrisbreiding and kuceb July 16, 2021 16:18
@flotwig flotwig marked this pull request as ready for review July 16, 2021 16:18
@flotwig flotwig requested a review from a team as a code owner July 16, 2021 16:18
Copy link
Contributor

@chrisbreiding chrisbreiding left a comment

Choose a reason for hiding this comment

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

Yay for deleted code 🎉

flotwig and others added 2 commits July 19, 2021 09:30
This was a workaround added to fix a bug in FF < 80. Since Cy removed support for FF < 86 in #17355, this is effectively dead code.

This does not remove the ability for users to manually run `Cypress.backend('firefox:force:gc')`.
@flotwig flotwig force-pushed the remove-ff-auto-gc branch from ad35833 to 159db71 Compare July 19, 2021 13:30
@flotwig flotwig changed the base branch from block-older-browsers to 8.0-release July 19, 2021 13:31
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.

2 participants