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

Add tests for gulp functionality #1312

Merged
merged 3 commits into from
May 17, 2022
Merged

Conversation

BenSurgisonGDS
Copy link
Contributor

@BenSurgisonGDS BenSurgisonGDS commented May 11, 2022

Add cypress to the CI, build integration tests to test the current gulp functionality with a view to maintain that functionality when gulp is replaced.

See Issue: #1307

@BenSurgisonGDS BenSurgisonGDS requested a review from a team as a code owner May 11, 2022 11:33
@BenSurgisonGDS BenSurgisonGDS linked an issue May 11, 2022 that may be closed by this pull request
4 tasks
@BenSurgisonGDS BenSurgisonGDS self-assigned this May 11, 2022
@BenSurgisonGDS BenSurgisonGDS force-pushed the add-tests-for-gulp-functionality branch 3 times, most recently from 307bc57 to abbe1d1 Compare May 11, 2022 17:45
@BenSurgisonGDS BenSurgisonGDS marked this pull request as draft May 12, 2022 10:04
@BenSurgisonGDS BenSurgisonGDS force-pushed the add-tests-for-gulp-functionality branch 2 times, most recently from 226be8f to 674594b Compare May 12, 2022 14:12
@lfdebrux lfdebrux force-pushed the add-tests-for-gulp-functionality branch from 070ddaa to a916056 Compare May 12, 2022 14:46
@BenSurgisonGDS BenSurgisonGDS force-pushed the add-tests-for-gulp-functionality branch from 0c61cfe to a16c2cf Compare May 13, 2022 13:50
@BenSurgisonGDS BenSurgisonGDS marked this pull request as ready for review May 13, 2022 14:04
@lfdebrux lfdebrux force-pushed the add-tests-for-gulp-functionality branch 3 times, most recently from 354d03c to 0a6168b Compare May 13, 2022 14:16
@BenSurgisonGDS BenSurgisonGDS force-pushed the add-tests-for-gulp-functionality branch from 8522a43 to e9c4612 Compare May 13, 2022 14:41
BenSurgisonGDS and others added 2 commits May 16, 2022 13:53
Currently the prototype kit is flaky on macOS, `gulp.watch` doesn't
always pick up changes to the assets folder. We're planning to rip gulp
out soon (#1290), so for now we just let them fail, and we should just
make sure to fix the flakiness then so we can remove this kludge.
@BenSurgisonGDS BenSurgisonGDS force-pushed the add-tests-for-gulp-functionality branch from cb83cd8 to 95d67d6 Compare May 16, 2022 12:56
@BenSurgisonGDS BenSurgisonGDS force-pushed the add-tests-for-gulp-functionality branch from 95d67d6 to 76720fa Compare May 16, 2022 12:57
@lfdebrux lfdebrux merged commit be89520 into main May 17, 2022
@lfdebrux lfdebrux deleted the add-tests-for-gulp-functionality branch May 17, 2022 08:52
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.

Add tests for Gulp functionality
3 participants