* fix(tools): implement better error handling
Modifies the workflow to validate the branch exists and the PR does
not. This was not blocking any issues, but caused the action to
report a failure.
* fix: apply Randy's review suggestions
Co-authored-by: Randell Dawson <5313213+RandellDawson@users.noreply.github.com>
Co-authored-by: Randell Dawson <5313213+RandellDawson@users.noreply.github.com>
* fix(tools): only camperbot translates
Modifies the no pr to translation workflow to fail on any PR that
was not created by camperbot.
* fix(tools): update message
* Update .github/workflows/no-prs-to-translation.yml
Co-authored-by: Randell Dawson <5313213+RandellDawson@users.noreply.github.com>
Co-authored-by: Randell Dawson <5313213+RandellDawson@users.noreply.github.com>
* fix(api): decouple api from curriculum
This reverts commit 8f0e441644 and
introduces the implementations from #40703.
* fix(gitpod): add curriculum build to GitPod
This reverts commit 706d70f58d and
introduces implementations from #41234.
* docs: update DevOps manual for api change (#41259)
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* fix(tools): adjust no-translate workflow
Modifies the no-prs-to-translation workflow to target our new `main`
branch, and verify that the PR comes from a forked repository.
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Co-authored-by: Randell Dawson <5313213+RandellDawson@users.noreply.github.com>
Co-authored-by: Mrugesh Mohapatra <1884376+raisedadead@users.noreply.github.com>
* fix(tools): allow partial translation
Modifies the workflow to allow for partial download of translations
to ensure we get all keys in the translation object even if they
still have english values.
NOTE: Sets workflow to dry run for testing purposes. Will need to
revert that setting.
* revert(tools): no dry run
Temporarily disables the automatic PR creation on Crowdin Download while
we determine the best approach for this. PRs created by the action
itself cannot trigger workflows, which prevents our CI from running. As
such, this is currently not a viable approach.
Signed-off-by: nhcarrigan <nhcarrigan@gmail.com>
* fix: adjust comment message
Changes the message this action uses for a PR comment to remove the
reference to the translation page pending announcement.
Signed-off-by: nhcarrigan <nhcarrigan@gmail.com>
* fix: expand comment
Signed-off-by: nhcarrigan <nhcarrigan@gmail.com>
* Update Cypress version from 5.3.0 to 6.0.0
* Replace `not.be.visible` with `not.exist`
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>