Skip to content

Conversation

@kwrooijen
Copy link
Contributor

@kwrooijen kwrooijen commented Jan 15, 2021

@kitallis
Copy link
Contributor

Hi @kwrooijen, thanks for the PR! Give me a day or so to go through this.

Copy link
Contributor

@scimetfoo scimetfoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Could you please add these to the index in the readme along with the examples?

README.md Outdated
`create-extension` can be used to create extensions with a given keyword.
```clj
(-> (create-extension :uuid-ossp :if-not-exists? true)
(sql/format :allow-dashed-names? true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you fix the formatting here please? Sorry for nitpicking.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops! my mistake

@scimetfoo scimetfoo merged commit 9a8cf11 into nilenso:master Feb 12, 2021
@kwrooijen kwrooijen deleted the feature/add-create-drop-extension branch February 12, 2021 12:50
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.

3 participants