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

Master to fork #1

Merged
merged 79 commits into from
Mar 4, 2019
Merged

Master to fork #1

merged 79 commits into from
Mar 4, 2019

Conversation

MiloTodt
Copy link
Owner

@MiloTodt MiloTodt commented Mar 4, 2019

Description

Issue fixed

Type of changes

  • ⚪ Bug fix (Change that fixed an issue)
  • ⚪ Breaking change (Change that can cause existing functionality to change)
  • ⚪ Improvement (Change that improves the code. Maybe performance or development improvement)
  • ⚪ Feature (Change that adds new functionality)
  • ⚪ Documentation change (Change that modifies documentation. Maybe typo fixes)

Checklist:

  • ⚪ My code follows the project code style
  • ⚪ I have written test for my code and it has been tested
  • ⚪ All existing tests have been passed
  • ⚪ I have attached a screenshot/video to visualize my change if possible

Rokt33r and others added 29 commits January 26, 2019 12:29
…d that issue in the future + test modified so that they both contain tests for posix and windows path separator -> fixes #2834
- improve test
- fix missing 'e' in some functions name
Strange url-handling reverted + tests modified so that they might fin…
fix image path due to bad regex
- Only strip the leading # in the title
- Make the finding title logic more straightforward
- Add unit test
@MiloTodt MiloTodt merged commit d6f30c3 into MiloTodt:master Mar 4, 2019
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.