Skip to content

Conversation

@msquance-stem
Copy link
Contributor

Status

Review progress:

  • Browser tested
  • Front-end review completed
  • Tech review completed

What's changed?

  • As content is now driven by Strapi, doing checks in the test environment, which pulls its data from staging doesnt make any sense
  • We have removed the lighthouse step of the CI pipeline

Steps to perform after deploying to production

If the production environment requires any extra work after this PR has been deployed detail it here. This could be running a Rake task, migrating a DB table, or upgrading a Gem. That kind of thing.

@sonarqubecloud
Copy link

@tc-deploybot tc-deploybot temporarily deployed to teachcomputing-pr-2368 March 20, 2025 11:18 Inactive
Copy link
Contributor

@A-Wheeto A-Wheeto left a comment

Choose a reason for hiding this comment

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

LGTM

@msquance-stem msquance-stem merged commit 63b81d9 into main Mar 21, 2025
5 checks passed
@msquance-stem msquance-stem deleted the 3017-remove-lighthouse-checks-from-ci branch March 21, 2025 09:18
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.

4 participants