Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Made more clear steps on the readme #128

Merged
merged 3 commits into from
Nov 25, 2024
Merged

Made more clear steps on the readme #128

merged 3 commits into from
Nov 25, 2024

Conversation

Emos21
Copy link
Collaborator

@Emos21 Emos21 commented Aug 14, 2024

I added more steps on the readme

README.md Outdated
```
PORT=9000
DATABASE_URL="postgresql://postgres:postgres@localhost:5432/claim"
ACCESS_SECRET=dPEBknfdAcx5bir34KnX2mATWZnvM4xF
Copy link
Collaborator

Choose a reason for hiding this comment

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

hey @Emos21 blank out the secrets in the pr, don't check in actual secrets - assume these are already compromised now so you should rotate them

Copy link
Collaborator

Choose a reason for hiding this comment

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

aside from that this is great


run from within an IDE such as webstorm with simple configuration such as

<img width="852" alt="image" src="https://user-images.githubusercontent.com/798887/232255771-e3cf52db-ece2-48b0-b67f-cd8edec39776.png">
Copy link
Collaborator

Choose a reason for hiding this comment

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

great this kind of thing is super helpful, we used to do this also at uber

@gvelez17
Copy link
Collaborator

ok great ,thanks, merging

@gvelez17 gvelez17 merged commit da983b7 into dev Nov 25, 2024
@gvelez17 gvelez17 deleted the amos branch November 25, 2024 04:28
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