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 :VimwikiNextTask #959

Merged
merged 1 commit into from
Jul 29, 2020
Merged

Fix :VimwikiNextTask #959

merged 1 commit into from
Jul 29, 2020

Conversation

rylwin
Copy link
Contributor

@rylwin rylwin commented Jul 29, 2020

Use the correct function for accessing 'rxListItemWithoutCB'.
ac4d0a1d46 refactored access to vars and it appears this is one place
where get_syntaxlocal should have been renamed to get_wikilocal but
was accidentally skipped.

Steps for submitting a pull request:

  • ALL pull requests should be made against the dev branch!
  • Take a look at CONTRIBUTING.MD
  • Reference any related issues.
  • Provide a description of the proposed changes.
  • PRs must pass Vint tests and add new Vader tests as applicable.
  • Make sure to update the documentation in doc/vimwiki.txt if applicable,
    including the Changelog and Contributors sections.

I've been running on dev to take advantage of some of the latest changes there and noticed that gnt no longer works for me. I think I tracked down the issue and added a test to confirm the bug and related fix. That said, this is my first time contributing to this project and using vader for tests. Happy to make any tweaks to get this PR cleaned up and ready to merge!

Use the correct function for accessing 'rxListItemWithoutCB'.
`ac4d0a1d46` refactored access to vars and it appears this is one place
where `get_syntaxlocal` should have been renamed to `get_wikilocal` but
was accidentally skipped.
@tinmarino
Copy link
Member

Hi @rylwin , nice catch, nice fix, nice tests.
Thank you.

@tinmarino
Copy link
Member

Please keep on with nice contribution like that: you can have a look at good first issue

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.

None yet

2 participants