-
Notifications
You must be signed in to change notification settings - Fork 75
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
Update data model #3350
Update data model #3350
Conversation
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #3350 +/- ##
==========================================
- Coverage 98.29% 98.20% -0.09%
==========================================
Files 292 287 -5
Lines 21415 20454 -961
==========================================
- Hits 21049 20087 -962
- Misses 366 367 +1 see 59 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
split data model update from fixture fix. add test to ensure fixtures are ok |
Makefile
Outdated
@@ -47,18 +47,18 @@ css: | |||
|
|||
%.pdf: | |||
mkdir -p docs/data-model | |||
postgresql_autodoc -h localhost -u geotrek -d geotrekdb -t dot -m "$*_.*" --password=geotrek -s "public" | |||
postgresql_autodoc -l /usr/share/postgresql-autodoc -h localhost -u geotrek -d geotrekdb -t dot -m "$*_.*" --password=geotrek -s "public" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
postgresql_autodoc -l /usr/share/postgresql-autodoc -h localhost -u geotrek -d geotrekdb -t dot -m "$*_.*" --password=geotrek -s "public" | |
postgresql_autodoc -l /usr/share/postgresql-autodoc -h localhost -u geotrek -d geotrekdb -t dot -m "$*_.*" --password=geotrek -s "public" |
Why using -l /usr/share/postgresql-autodoc
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no need to specify on my instance as it's default shared data for postgresql-autodoc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, on ubuntu 22.04 ?
I installed autodoc especially for this purpose and by default it seems that the program was trying to reach another folder (as mentioned in this issue cbbrowne/autodoc#26).
If this is only a bug on my computer I guess we should remove this commit then.
0c3d196
to
cdc4e55
Compare
No description provided.