-
Notifications
You must be signed in to change notification settings - Fork 22
chore/update #589
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
chore/update #589
Conversation
| if [ ! -f "/home/vscode/.local/share/pg/pgdata/PG_VERSION" ]; then | ||
| echo "Initializing Postgres..." | ||
| /usr/lib/postgresql/16/bin/initdb -D /home/vscode/.local/share/pg/pgdata --auth trust --auth-local trust --auth-host trust | ||
| /usr/lib/postgresql/16/bin/initdb -D /home/vscode/.local/share/pg/pgdata --auth trust --auth-local trust --auth-host trust --encoding=UTF8 --locale=C.UTF-8 |
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.
this got clobbered again by some hook magic I didn't fully remove, sneaking in here b/c it was breaking my tests for validating the upgrades
- need rclone >= v1.68.0 to upgrade away from deprecated v1 aws sdk - need go-libp2p >= v0.44.0 for go-quic version compatibility - rclone dropped fs.NewConfig(), need to hardcode more defaults :( - azureblob retired years ago
lanzafame
left a comment
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.
excellent clean up
|
hmmm we are still skipping some tests that I would expect to run, I think there's an issue with the dialector for sqlite bailing and skipping the other tests; I'll look into that to validate the FK/cascade fix properly but otherwise we should be good |
|
ok I think the skips are reasonable dialector mismatch (postgres specific test should not run with mysql etc) so we should be good |
|
interesting, the glif-dependent tests fail on |
|
breakage fixed in #591 |

omnibus PR to update our many, many badly outdated dependencies
major changes: