-
Notifications
You must be signed in to change notification settings - Fork 98
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
Pull request 487 fix #491
Pull request 487 fix #491
Conversation
@hbelmiro I wasn’t sure if I should wait for @ricardozanini first, so just to avoid any misunderstanding, I created the PR. Please let me know if there's something else I can help with. |
Sh*t, I was wondering why I got conflicts and I just realized, I didn't create the PR from |
@patr1kt0th It's not a problem if it's effectively changing what it should change. But if you want to fix it anyway, you can just rebase your branch. |
* Add @PartFilename config - on/off, value, value suffix * Split PartFilenameTest into classic and reactive * Get rid of code duplicity in tests, fix README.md * Revert messed up README.md formatting * Remove problematic dependencies, add testExcludes for Classic/Reactive tests
* Add @PartFilename config - on/off, value, value suffix * Split PartFilenameTest into classic and reactive * Get rid of code duplicity in tests, fix README.md * Revert messed up README.md formatting * Remove problematic dependencies, add testExcludes for Classic/Reactive tests Co-authored-by: Patrik Toth <[email protected]>
Fix for #487