Skip to content

Conversation

@coalest
Copy link
Collaborator

@coalest coalest commented Mar 20, 2025

(Just a minor nit of a PR)

Description

Fixes the following error message that occurs on app startup in dev:
cat: ./REVISION: No such file or directory

That file is created in prod by Cloud 66, but not in dev/test hence the warning/error message.

Type of change

  • Internal Refactor

How Has This Been Tested?

Manually. See screenshots

Screenshots

Before the change:
Screenshot from 2025-03-20 12-50-04

After the change (first without the REVISION file like in non-prod environments and then with the file):
Screenshot from 2025-03-20 12-48-06

@cielf cielf requested a review from awwaiid March 20, 2025 12:56
Copy link
Collaborator

@awwaiid awwaiid left a comment

Choose a reason for hiding this comment

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

This works. I had wondered about putting DEVELOPMENT as the content of the file and commit it, but I think this is better (and no risk of cloud66 then getting mad at the file existing)

@coalest
Copy link
Collaborator Author

coalest commented Mar 20, 2025

btw CI/CD won't pass until this flaky test gets fixed #5108

@awwaiid awwaiid merged commit 42fe2bd into rubyforgood:main Mar 20, 2025
11 checks passed
@github-actions
Copy link
Contributor

@coalest: Your PR Fix cat REVISION file warning on app startup is part of today's Human Essentials production release: 2025.03.23.
Thank you very much for your contribution!

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.

2 participants