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.

Sign up for GitHub

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

Jump to bottom

Validate mirror: remove no-pub create check #1307

Merged

serprex merged 1 commit into main from validation/remove-pub-check

Feb 15, 2024

Contributor

Amogh-Bharadwaj commented Feb 15, 2024

There isn't a universal way to check for superuser across managed postgres offerings, making permission checking infeasible
Not ideal to have the risk of having a stray publication lying around if we go with the create-drop dummy publication route


          remove no-pub create check

4b659c4

serprex approved these changes

View reviewed changes

serprex requested a review from iskakaushik

February 15, 2024 21:41

serprex merged commit 790d305 into main

7 checks passed

serprex deleted the validation/remove-pub-check branch

February 15, 2024 21:42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet