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

Config with pure python #12

Merged
merged 5 commits into from
Nov 16, 2021
Merged

Config with pure python #12

merged 5 commits into from
Nov 16, 2021

Conversation

icemac
Copy link
Member

@icemac icemac commented Nov 12, 2021

Run the tests on GHA.

Locally on MacOS they fail miserably. But maybe #10 could help here.

@icemac icemac self-assigned this Nov 12, 2021
@icemac
Copy link
Member Author

icemac commented Nov 12, 2021

Rebased after #10 was merged to master.

@navytux navytux mentioned this pull request Nov 12, 2021
@icemac
Copy link
Member Author

icemac commented Nov 15, 2021

Interestingly the tests against ZODB 4 run fine.
@navytux Do you think you are able to also fix the remaining broken tests?

@navytux
Copy link
Contributor

navytux commented Nov 15, 2021

@icemac, thanks for asking. I believe yes: please have a look at #13.

@icemac icemac marked this pull request as ready for review November 16, 2021 07:19
@icemac icemac requested a review from navytux November 16, 2021 07:21
@navytux
Copy link
Contributor

navytux commented Nov 16, 2021

@icemac, this PR looks ok to me.
Thanks for caring about zc.zlibstorage state.

@icemac icemac merged commit 8df6c70 into master Nov 16, 2021
@icemac icemac deleted the config-with-pure-python branch November 16, 2021 08:00
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.

2 participants