Skip to content

Minor fixes#172

Closed
anjiro wants to merge 4 commits intolervag:masterfrom
anjiro:master
Closed

Minor fixes#172
anjiro wants to merge 4 commits intolervag:masterfrom
anjiro:master

Conversation

@anjiro
Copy link
Copy Markdown
Contributor

@anjiro anjiro commented Jul 17, 2021

A small batch of minor fixes.

anjiro added 4 commits July 17, 2021 22:55
`l:origin[1]` is the results of `getpos(.)`, not just a line number,
causing an error on template rendering
let l:level = len(matchstr(l:line, '^#*'))
let l:header = matchlist(l:line, g:wiki#rx#header_items)[2]
if l:header ==# 'Innhald' | continue | endif
if l:header ==# g:wiki_toc_title | continue | endif
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, sorry! And thanks!

\ 'name': expand('%:t:r'),
\ 'origin_file': l:origin[0],
\ 'origin_lnum': l:origin[1],
\ 'origin_lnum': l:origin[1][1],
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! I'll merge and then also fix a related problem in nav.vim.


*g:wiki_cache_persistent*
Specify whether to use persistant caching.
Specify whether to use persistent caching.
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've been caught misspelling this several times now. I should probably learn how to spell...

lervag added a commit that referenced this pull request Jul 17, 2021
lervag added a commit that referenced this pull request Jul 17, 2021
@lervag
Copy link
Copy Markdown
Owner

lervag commented Jul 17, 2021

Thanks! I've merged this now and also fixed a minor consistency error I noticed while reviewing.

@lervag lervag closed this Jul 17, 2021
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.

2 participants