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

Avoid Markdown-folding in diff mode #538

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Avoid Markdown-folding in diff mode #538

wants to merge 7 commits into from

Conversation

KSR-Yasuda
Copy link
Contributor

Fix for #427.

@KSR-Yasuda
Set Markdown foldmethod if no foldexpr is set
10e9c75 
- To avoid Markdown-folding in diff mode
- Known issue: If any other `foldexpr` is already set precedingly, Markdown-folding is suppressed
@KSR-Yasuda
Test fix
23a00a2
@KSR-Yasuda
Fix foldexpr default value assumption
b6637cc
@KSR-Yasuda
Revert "Test fix"
242dc6e 
This reverts commit 23a00a2.
@KSR-Yasuda
Allow empty foldexpr too for Markdown-folding alongside the default…
f273a6c 
… value `0`
@KSR-Yasuda
Copy link
Contributor Author

Off-topic: some tests do not work correctly in non-English environment?

For example, fold text result test in test/folding-toc.vader,
it expects +-- 28 lines: hello in the folding marker line, though,
Japanese environment shows +-- 28 行: hello, and the test fails.

Tried to fix it with setting $LANG, $LANGUAGE, and $LC_ALL environment variables in test/run-tests.sh,
but it did not have any effect.
I gave up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@KSR-Yasuda