Skip to content

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