Update ERD to reflect current DB structure #3032
Conversation
|
Maybe we can remove it since it is not easy to keep it updated. Or we can add it to CI to compare it is at latest version. |
I really liked having it linked in the contribution guidelines as someone who hasn't contributed to this repo before. It was helpful even if it was out of date. So I like the idea of adding it as a CI step. |
|
OK, I think you can add additional "ERD" workflow started only on |
|
Can you temporarily allow this branch on CI as well? Also CI is ubuntu based, brew is not available in there. |
|
I looked at their docs and it seemed that Homebrew was available And sure, I'll add this branch to the filter |
|
It is better to use 'apt-get install'. |
|
Even though I allowed this branch, the new action won't run b/c the schema hasn't changed. |
|
You can remove the paths setting for now. |
|
Looks good to me. Feel free to add back the paths and I think we can merge this. 💪 |
|
Can you squash commits please? |
|
Ahh, I just realized, can you add |
Gladly, do you know where I do that? |
The ERD hasn't been updated in about 2.5 years, updating to reflect changes Generated with `rake gen_erd` Added Github Action to check for latest ERD on db/schema.rb change in push to master or PR
|
Thanks for your patience. I'm going to 🙏 for the CI and merge. |
The ERD hasn't been updated in about 2.5 years, updating to reflect changes
Generated with
rake gen_erd(note: you need to install Graphviz to run thisbrew install graphviz)