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

Allow partitioned tables to be returned by PgSqlDriver::getTables() #286

Closed
wants to merge 1 commit into from

Conversation

otuhacek
Copy link
Contributor

@otuhacek otuhacek commented Dec 9, 2021

  • bug fix? yes
  • BC break? no

@diegosardina
Copy link
Contributor

Could you put 'p' as the last item? Just to follow the order specified in the documentation

@milo milo self-assigned this Dec 10, 2021
milo pushed a commit to milo/nette-database that referenced this pull request Dec 10, 2021
@milo
Copy link
Member

milo commented Dec 10, 2021

Hi @otuhacek . I improved your commit a little bit (see #287) and kept your authority if you are OK with it.

milo pushed a commit to milo/nette-database that referenced this pull request Dec 10, 2021
@dg dg force-pushed the master branch 2 times, most recently from f6be0c8 to f534aa7 Compare December 12, 2021 03:35
dg pushed a commit that referenced this pull request Dec 12, 2021
@dg
Copy link
Member

dg commented Dec 12, 2021

Fixed by #287

@dg dg closed this Dec 12, 2021
dg pushed a commit that referenced this pull request Dec 12, 2021
dg pushed a commit that referenced this pull request Dec 12, 2021
dg pushed a commit that referenced this pull request Dec 12, 2021
dg pushed a commit that referenced this pull request Dec 12, 2021
dg pushed a commit that referenced this pull request Dec 12, 2021
dg pushed a commit that referenced this pull request Jan 19, 2022
dg pushed a commit that referenced this pull request Jan 19, 2022
dg pushed a commit that referenced this pull request Jan 19, 2022
dg pushed a commit that referenced this pull request Jan 19, 2022
dg pushed a commit that referenced this pull request Mar 10, 2022
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.

4 participants