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: decaffeinate server tests w/in fixtures/projects (not e2e) #9154

Merged
merged 4 commits into from
Dec 4, 2020

Conversation

jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Nov 10, 2020

Decaffeinating some of the leftover coffeescript files in the server tests

DO NOT SQUASH

DO NOT MERGE IN DEVELOP

These kitchensink tests are failing from 6.0 cy.intercept changes. I don't want to merge in develop because it always messes up the commits on decaffeinate files, so ignore the failures.

@cypress-bot
Copy link
Contributor

cypress-bot bot commented Nov 10, 2020

Thanks for taking the time to open a PR!

@cypress
Copy link

cypress bot commented Nov 10, 2020



Test summary

3929 0 49 2


Run details

Project cypress
Status Passed
Commit 13580a8
Started Dec 4, 2020 2:40 AM
Ended Dec 4, 2020 2:50 AM
Duration 10:10 💡
OS Linux Debian - 10.5
Browser Chrome 83

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

@jennifer-shehane jennifer-shehane marked this pull request as draft November 17, 2020 06:04
@jennifer-shehane jennifer-shehane force-pushed the remove-remaining-coffee-tests-2 branch 4 times, most recently from ed89c13 to 7518e8d Compare November 18, 2020 09:43
@jennifer-shehane jennifer-shehane marked this pull request as ready for review December 3, 2020 05:09
@jennifer-shehane
Copy link
Member Author

These kitchensink tests are failing from 6.0 cy.intercept changes. I don't want to merge in develop because it always messes up the commits on decaffeinate files, so ignore the failures.

@kuceb
Copy link
Contributor

kuceb commented Dec 3, 2020

force pushed to remove unneeded eslint-disable messages

kuceb
kuceb previously approved these changes Dec 3, 2020
cleanup js file references

Convert missed coffee file

Cleanup js

Cleanup js

Clean up js files

Clean up js files

Clean up js files

Clean up js files
@kuceb
Copy link
Contributor

kuceb commented Dec 3, 2020

rebased on develop

@jennifer-shehane jennifer-shehane merged commit a5c8182 into develop Dec 4, 2020
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

2 participants