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

Port test-tutorial script to JS #1896

Merged
merged 14 commits into from Mar 4, 2021
Merged

Conversation

dac09
Copy link
Collaborator

@dac09 dac09 commented Mar 2, 2021

What?

As part of running e2e tests in CI, this PR ports test-tutorial bash script to javascript, to make it more cross-platform compatible

Related to #1895

Working towards #1886

@dac09 dac09 marked this pull request as ready for review March 2, 2021 16:12
@dac09 dac09 requested a review from peterp March 2, 2021 16:13
@dac09
Copy link
Collaborator Author

dac09 commented Mar 2, 2021

Skipping changing the linking to rwt link, because it actually makes the code a lot more complicated!

tasks/test-tutorial.js Outdated Show resolved Hide resolved
tasks/test-tutorial.js Outdated Show resolved Hide resolved
@dac09 dac09 added this to the next release milestone Mar 2, 2021
@dac09 dac09 mentioned this pull request Mar 2, 2021
1 task
@thedavidprice thedavidprice self-requested a review March 2, 2021 22:26
Copy link
Contributor

@peterp peterp left a comment

Choose a reason for hiding this comment

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

LGTM. Some stylistic suggestions.

.eslintrc.js Outdated Show resolved Hide resolved
tasks/test-tutorial.js Show resolved Hide resolved
tasks/test-tutorial.js Outdated Show resolved Hide resolved
tasks/test-tutorial.js Outdated Show resolved Hide resolved
@github-actions
Copy link

github-actions bot commented Mar 4, 2021

📦 PR Packages

Click to Show Package Download Links

https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/create-redwood-app-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-api-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-api-server-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-auth-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-cli-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-core-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-dev-server-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-eslint-config-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-eslint-plugin-redwood-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-forms-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-internal-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-prerender-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-router-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-structure-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-testing-0.26.2-d819a06.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/1896/redwoodjs-web-0.26.2-d819a06.tgz

Install this PR by running yarn rw upgrade --pr 1896:0.26.2-d819a06

Copy link
Contributor

@thedavidprice thedavidprice left a comment

Choose a reason for hiding this comment

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

This reverts the desired behavior when providing a path to an existing project to use the project's installed packages — currently, it links and uses branch packages.

I'll take a look at the code to see if I can adjust.

@dac09
Copy link
Collaborator Author

dac09 commented Mar 4, 2021

This reverts the desired behavior when providing a path to existing project to use the installed packages — not to link and use branch packages.

@thedavidprice gotcha! Pushing up fix 2 mins :)

@thedavidprice thedavidprice merged commit df8d0c2 into redwoodjs:main Mar 4, 2021
@dac09 dac09 deleted the e2e/port-test-2-js branch March 5, 2021 00:03
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

4 participants