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

RawSql instances for 9, 10, 11 and 12-column results #961

Merged
merged 4 commits into from
Oct 14, 2019
Merged

RawSql instances for 9, 10, 11 and 12-column results #961

merged 4 commits into from
Oct 14, 2019

Conversation

dgaw
Copy link
Contributor

@dgaw dgaw commented Sep 26, 2019

Adds extra RawSql instances discussed in #835

@parsonsmatt
Copy link
Collaborator

parsonsmatt commented Oct 11, 2019

Looks great, thanks!

Can you add @since 2.10.2 to these functions and instances as well as a ChangeLog entry for this?

The syntax for the doc comment is like:

-- | 
-- @since 2.10.2
to10 :: -- ...

@parsonsmatt parsonsmatt added this to the 2.10.2 milestone Oct 11, 2019
@dgaw
Copy link
Contributor Author

dgaw commented Oct 14, 2019

@parsonsmatt Thanks for reviewing. I've added the since comments and the changelog entry.

Copy link
Collaborator

@parsonsmatt parsonsmatt left a comment

Choose a reason for hiding this comment

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

awesome thanks! Just one change and we're good :D

persistent/ChangeLog.md Outdated Show resolved Hide resolved
Co-Authored-By: Matt Parsons <parsonsmatt@gmail.com>
@parsonsmatt parsonsmatt merged commit db2e667 into yesodweb:master Oct 14, 2019
@dgaw dgaw deleted the extra-rawsql-instances branch October 15, 2019 16:10
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.

None yet

2 participants