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 Issue #726 - Adding a "folder does not have filter" #772

Merged
merged 1 commit into from
Oct 31, 2022

Conversation

rokaicker
Copy link
Contributor

@rokaicker rokaicker commented Oct 8, 2022

Hi!

I think I've got the new feature working (Issue 726). Please let me know if there are any changes to be made.

Also, would it be okay to tag this for hacktoberfest?

By the way, your code base is amazingly organized and incredibly modular, and I learned a lot just by navigating through the filter workflow.

@whyboris
Copy link
Owner

Thank you for the contribution! 🙇
I'll try to find time this weekend to test it locally 🤝
From my glance at the code - it looks good! 😁

@rokaicker
Copy link
Contributor Author

Learned about git rebase this week, so I thought it would be best for cleanliness if I combined all my commits into a single one.

@whyboris
Copy link
Owner

I'm a big fan of Squash and merge - so whenever I merge a PR, no matter how many commits it contains, the resulting merge shows 1 commit with the PR number. This makes the commit history very clean 😎

@rokaicker
Copy link
Contributor Author

Ooo! I didn't know that was possible, that also seems like a very useful tool from a maintainer perspective! I'm still learning so many new things about git, thanks!

@whyboris
Copy link
Owner

Thank you @rokaicker for your PR 🙇

@whyboris whyboris merged commit 354cea4 into whyboris:main Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants