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

Tweak initial setup instructions #1

Merged
merged 1 commit into from
Apr 23, 2024
Merged

Tweak initial setup instructions #1

merged 1 commit into from
Apr 23, 2024

Conversation

tdooner
Copy link
Contributor

@tdooner tdooner commented Apr 23, 2024

Tweak setup instructions for iv-cbv-payroll

  • Use rbenv and nodenv to ensure we're all using consistent versions
    of Ruby & Node, respectively.
  • Add instructions to start postgresql & redis
  • Check in generated ER Diagram, which it seems like we should do every
    time we change our database schema

* Use `rbenv` and `nodenv` to ensure we're all using consistent versions
  of Ruby & Node, respectively.
* Add instructions to start postgresql & redis
* Check in generated ER Diagram, which it seems like we should do every
  time we change our database schema
@tdooner tdooner requested a review from allthesignals April 23, 2024 17:59
@allthesignals
Copy link
Contributor

allthesignals commented Apr 23, 2024

Oh interesting, is this the ER diagram? https://github.com/DSACMS/iv-cbv-payroll/blob/57fd977564810c94e161f46c9344ead5e20e1521/doc/compliance/rendered/apps/data.logical.pdf what is ER? Googled: Entity-Relationship Model! Okay!

@tdooner
Copy link
Contributor Author

tdooner commented Apr 23, 2024

Going to merge so I can troubleshoot the tests failing in a different PR. LMK if you have feedback @allthesignals

@tdooner tdooner merged commit 61450d4 into main Apr 23, 2024
3 of 7 checks passed
@tdooner tdooner deleted the inital-setup-tweaks branch April 23, 2024 22:01
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