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

Setting up tests

Most tests will run automatically when you open a pull request, while others require contributors with write access to enable.
Until all tests pass, a pull request will not be merged.

enable instructions tbd

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 83-add-linting-to-github-test-automation [email protected]:platformsh/demo-project.git demo-test-83-add-linting-to-github-test-automation && cd demo-test-83-add-linting-to-github-test-automation

2. Create a project on Upsun:

   upsun project:create --title "Demo Project test (pr-84/merge)" --default-branch 83-add-linting-to-github-test-automation

   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 - I'm going to pause on this one to reduce noise to the commit log. Feel free to update the code if you know what the issue is.

Locally, the new linting and prettier commands work as expected.

While running, the GitHub checks are (1) failing to use the Python packages installed by npm install and (2) they keep aborting with "This check was canceled".

Once the .github/workflows/test.yaml is worked out, this MR is ready.

@tylers-username Based on the error, I would guess we need to add activate the venv to use black.

For example - pip-audit in test.sh

@chadwcarlson - I was going to sneak some corrections into the Upsun DEMO CI/demo-runthrough test, but I don't have permission to manage the project that is being created.

I think the biggest step is implementing a 28 character limit on branch names and project titles for Upsun within these tests.

@chadwcarlson - I was going to sneak some corrections into the Upsun DEMO CI/demo-runthrough test, but I don't have permission to manage the project that is being created.

I think the biggest step is implementing a 28 character limit on branch names and project titles for Upsun within these tests.

@chadwcarlson I restored .github/workflows/project.yaml to the state that we currently have in the main branch. This MR is ready for review.