Skip to content

Fix read-only mode#16

Open
sylvinus wants to merge 1 commit into
jamesls:masterfrom
pricingassistant:master
Open

Fix read-only mode#16
sylvinus wants to merge 1 commit into
jamesls:masterfrom
pricingassistant:master

Conversation

@sylvinus

Copy link
Copy Markdown

In read-only mode, open data file as read-only to avoid crash on read-only filesystems

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.2%) to 92.903% when pulling b78dd32 on pricingassistant:master into adcd3b2 on jamesls:master.

@besfahbod

Copy link
Copy Markdown

We have a dependency on this module which is now affected by this problem.

@jamesls, any chance you have the time to merge and make a release? If no, would it be possible to give @semidbm (or me) access to do so?

@besfahbod

Copy link
Copy Markdown

See also #18 .

@oxij

oxij commented Feb 6, 2020

Copy link
Copy Markdown

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