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

Pull request 487 fix #491

Merged
merged 6 commits into from
Sep 22, 2023
Merged

Conversation

patr1kt0th
Copy link
Contributor

Fix for #487

@patr1kt0th
Copy link
Contributor Author

@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.

@patr1kt0th
Copy link
Contributor Author

Sh*t, I was wondering why I got conflicts and I just realized, I didn't create the PR from main (I thought I switched before creating the branch, but I didn't), so if that's a problem, I can delete this one and create another PR.

@hbelmiro
Copy link
Contributor

@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.

@hbelmiro hbelmiro merged commit f1f34ed into quarkiverse:main Sep 22, 2023
github-actions bot pushed a commit that referenced this pull request Sep 22, 2023
* 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
hbelmiro pushed a commit that referenced this pull request Sep 22, 2023
* 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]>
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.

3 participants