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

Add option to disable creation of mermaid-filter.err #87

Open
CmdrSharp opened this issue Jun 7, 2022 · 4 comments
Open

Add option to disable creation of mermaid-filter.err #87

CmdrSharp opened this issue Jun 7, 2022 · 4 comments

Comments

@CmdrSharp
Copy link

Currently, mermaid-filter fails on read-only filesystems (which is not uncommon in Docker, for example).
This log file should, I feel, be optional. Along with that, perhaps an argument to specify a path for it would be good?

@chasecaleb
Copy link

It would be particularly nice to avoid creating this file when there aren't any errors.

@raghur
Copy link
Owner

raghur commented Oct 1, 2022

Can you pls try this branch out? tq

@chasecaleb
Copy link

@raghur Hi, so sorry I just realized you're probably waiting for a response from me. Unfortunately I don't have a straightforward way to test out a branch that isn't published on NPM. When you merge and publish NPM then I'm happy to try it out, though.

I skimmed the PR and it looks sane to me, if that's any help. Sorry about that.

@ghosty141
Copy link

I'd prefer a mode that just doesn't write any files at all.

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 a pull request may close this issue.

4 participants