]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commit
Include PostgreSQL in trigger test and correct documentation example
authorRamonWill <ramonwilliams@hotmail.co.uk>
Sun, 30 Aug 2020 13:05:06 +0000 (09:05 -0400)
committerRamonWill <ramonwilliams@hotmail.co.uk>
Mon, 31 Aug 2020 15:13:29 +0000 (16:13 +0100)
commit2e887c7a4105a973a015b9ecbb4ea5a11dafb26a
treea8af4588ad1127b943847e4c8c70228a3efe859c
parent34dfd7eb88f42e259d3049dbbc823a97b11cb555
Include PostgreSQL in trigger test and correct documentation example

Include PostgreSQL dialect in trigger test and correct DDL example in documentation

A user highlighted that the syntax in the DDL trigger example was incorrect for PostgreSQL. The trigger tests where also skipping the PostgreSQL dialect until the syntax was corrected. This PR fixes both of these issues.

This pull request is:

- [X ] A documentation / typographical error fix
- Good to go, no issue or tests are needed
- [ ] A short code fix
- please include the issue number, and create an issue if none exists, which
  must include a complete example of the issue.  one line code fixes without an
  issue and demonstration will not be accepted.
- Please include: `Fixes: #<issue number>` in the commit message
- please include tests.   one line code fixes without tests will not be accepted.
- [ ] A new feature implementation
- please include the issue number, and create an issue if none exists, which must
  include a complete example of how the feature would look.
- Please include: `Fixes: #<issue number>` in the commit message
- please include tests.

**Have a nice day!**
Fixes: #4037
Closes: #5548
Pull-request: https://github.com/sqlalchemy/sqlalchemy/pull/5548
Pull-request-sha: 1db5e47adb90f9d51e247711dcfdbb274fb7bf73

Change-Id: I06edbcab99c82a3ce25581b81f8d2a4a028c07c3
doc/build/core/ddl.rst
test/orm/test_defaults.py
test/requirements.py