Skip to content

Issue 67#68

Merged
silenius merged 18 commits into
zopefoundation:masterfrom
silenius:issue_67
Jul 13, 2021
Merged

Issue 67#68
silenius merged 18 commits into
zopefoundation:masterfrom
silenius:issue_67

Conversation

@silenius
Copy link
Copy Markdown
Contributor

@silenius silenius commented Jun 29, 2021

This is a fix for #67 with changes from #66 included

Close #66.

Comment thread src/zope/sqlalchemy/datamanager.py Outdated
@icemac
Copy link
Copy Markdown
Member

icemac commented Jul 2, 2021

All tests are breaking on GHA. On master at least Python 3.6+ and lint are successful. Could you please have a look?

@silenius
Copy link
Copy Markdown
Contributor Author

silenius commented Jul 2, 2021

it should be OK for >3.5 (tests are broken for < 3.6 (as you noticed in #69))

@icemac
Copy link
Copy Markdown
Member

icemac commented Jul 2, 2021

@silenius The lint checks still fail. For Python < 3.6 I am preparing a fix at #70.

@icemac
Copy link
Copy Markdown
Member

icemac commented Jul 2, 2021

#70 is now merged, I added the changes to this PR, too.

@icemac
Copy link
Copy Markdown
Member

icemac commented Jul 12, 2021

Thank you for your contribution.

According to the contributing policies of the zopefoundation organization you need to sign a contributor agreement before any non-trivial change can be merged. For details please consult the Contributing guidelines for zopefoundation projects.

@silenius
Copy link
Copy Markdown
Contributor Author

Thank you for looking at this. I'll check the contributor agreement document as soon as I have a little time and keep you updated..

@silenius
Copy link
Copy Markdown
Contributor Author

I've completed and sent the PDF document to agreements@plone.org

Copy link
Copy Markdown
Member

@icemac icemac left a comment

Choose a reason for hiding this comment

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

LGTM.

In zopefoundation repositories the policy is that the creator of the PR merges it, to prove having signed the contributor agreement.

@silenius silenius merged commit b65543f into zopefoundation:master Jul 13, 2021
@silenius
Copy link
Copy Markdown
Contributor Author

✔️

@silenius silenius deleted the issue_67 branch July 13, 2021 07:57
@icemac
Copy link
Copy Markdown
Member

icemac commented Jul 14, 2021

I am going to write a change log entry and to cut a release now.

@icemac
Copy link
Copy Markdown
Member

icemac commented Jul 14, 2021

I just released https://pypi.org/project/zope.sqlalchemy/1.5/

@icemac icemac mentioned this pull request Jul 14, 2021
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