Skip to content

Conversation

@melange396
Copy link
Contributor

The "ci" gh action had a failure in the lint step after a recent PR (which failed the "prettier" check) was merged.

The failure was not noticed before merge because the check was not even run, as the PR was based on a forked branch instead of the typical branch inside this repo.

(this change is very much like the one in https://github.com/cmu-delphi/www-epivis/pull/109/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R27 )

@netlify
Copy link

netlify bot commented Jun 12, 2025

Deploy Preview for cmu-delphi-main ready!

Name Link
🔨 Latest commit 64c3d17
🔍 Latest deploy log https://app.netlify.com/projects/cmu-delphi-main/deploys/684a385eba8de30008915c73
😎 Deploy Preview https://deploy-preview-1095--cmu-delphi-main.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@melange396
Copy link
Contributor Author

The output of prettier is already available:
https://github.com/cmu-delphi/www-main/actions/runs/15599722157/job/43937303735#step:6:112

i will now make the change to pass linting

@melange396 melange396 merged commit db7a3d5 into dev Jun 12, 2025
7 checks passed
@melange396 melange396 deleted the enable_prettier_output branch June 12, 2025 15:56
@melange396 melange396 mentioned this pull request Jun 23, 2025
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.

3 participants