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

Fix cache's file permission to 600 #318

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Yuki-Inoue
Copy link

Since cache includes private information, (the auth-info), it's
permission should be set to 600.

Since cache includes private information, (the auth-info), it's
permission should be set to 600.
@vermiculus
Copy link
Owner

Change looks good, thanks :) but it seems our tests need updating…

@Malabarba @jrblevin Has anything changed with markdown processing that could cause indented space after a code block?

i.e.,

screen shot 2016-05-15 at 9 52 17 am

@Malabarba
Copy link
Collaborator

We could probably just change our renderer to delete-trailing-whitespace after everything is done rendering.

@jrblevin
Copy link

I see that the text in the screenshot is from a test case. Should I think of it as pre-existing text that is fontified using markdown-mode, or is it being inserted and/or filled using markdown-mode functions? In other words, how does one reproduce this in a markdown-mode buffer?

@vermiculus
Copy link
Owner

I'm going to revamp the tests separately; we'll see how things fall after that.

@vermiculus
Copy link
Owner

I never did get around to revamping tests. Is this something that's still valuable to merge after all this time? Happy to do so if so.

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