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

Remix: remove backend app to simplify. #262

Merged
merged 15 commits into from
Oct 16, 2024
Merged

Remix: remove backend app to simplify. #262

merged 15 commits into from
Oct 16, 2024

Conversation

chadwcarlson
Copy link
Contributor

Copy link

Hello, and thank you for contributing to the Upsun demo project!

Setting up tests

Until all tests pass, a pull request will not be merged.

Build tests for combinations of Python and Node.js will happen automatically, and must pass in order to merge.
Another test will actually runthrough all steps of the demo on a new project, but requires a repository admin to add the runthrough label to the pull request.

Failing tests

If your PR is failing, see the Running tests section of the README for troubleshooting details.

Reviewing these changes locally:

  1. Clone this repo

    git clone -b remixed [email protected]:platformsh/demo-project.git demo-test-remixed && cd demo-test-remixed
  2. Create a project on Upsun:

    upsun project:create --title "Demo Project test (pr-262/merge)" --default-branch remixed

    This command will set the local repository remote automatically, but you will need to specify the organization and region.

  3. Push to Upsun:

    upsun push

@chadwcarlson chadwcarlson added enhancement New feature or request runthrough and removed runthrough labels Apr 19, 2024
@chadwcarlson chadwcarlson merged commit 7e7e9c8 into main Oct 16, 2024
4 checks passed
@chadwcarlson chadwcarlson deleted the remixed branch October 16, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request runthrough
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant