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.

Sign up for GitHub

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

Jump to bottom

python: handle open modes explicitly #177

Merged
merged 4 commits into from
Oct 11, 2024
Merged

python: handle open modes explicitly #177

merged 4 commits into from
Oct 11, 2024

Conversation

paraseba
Copy link
Contributor

Also better error message when trying to open an existing store and it doesn't exist

@paraseba
python: handle open modes explicitly
c94f2e7 
Also better error message when trying to open an existing store and it
doesn't exist
@paraseba paraseba requested review from rabernat and jhamman October 10, 2024 14:51
@paraseba
Copy link
Contributor Author

@jhamman could you check if this fixes #169 ?

@paraseba paraseba requested review from mpiannucci and removed request for rabernat October 10, 2024 15:15
@paraseba paraseba merged commit ae2d793 into main Oct 11, 2024
2 checks passed
@paraseba paraseba deleted the seba/open-mode branch October 11, 2024 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mpiannucci mpiannucci mpiannucci approved these changes

@jhamman jhamman Awaiting requested review from jhamman

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@paraseba @mpiannucci