Skip to content

Fixed ignoring routes #20

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
Merged
merged 4 commits into from
May 7, 2020
Merged

Fixed ignoring routes #20

merged 4 commits into from
May 7, 2020

Conversation

@Fifciu
Copy link

@Fifciu Fifciu commented Feb 3, 2020

Fixed it: #19
Close #19
@Fifciu
Copy link
Author

Fifciu commented Feb 3, 2020

Also added: leaveIgnored option. So it is possible to do not affect ignored routes (not setting y: 0)
Fifciuu added 2 commits February 3, 2020 14:27
@Fifciu Fifciu requested a review from jeneser February 9, 2020 13:40
@jeneser
Copy link
Owner

jeneser commented May 7, 2020

Thank you for the PR.
jeneser
jeneser reviewed May 7, 2020
View reviewed changes
Copy link
Owner

@jeneser jeneser left a comment

Choose a reason for hiding this comment

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

Good job!
@jeneser jeneser added the bug label May 7, 2020
@jeneser jeneser merged commit 4950969 into jeneser:master May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug

2 participants

@Fifciu @jeneser