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

Testcontainers integration #21

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

auriium
Copy link
Contributor

@auriium auriium commented Mar 15, 2021

the fucking modularization keeps breaking it thinks that the sources don't exist even when they do

Theyre being generated on phase generate-sources and are being put into the folder generate sources what else does it even want

Besides these fucking issues are more issues in that i'm eventually going to have to compeltely tear out and replace testcontainers with docker (which a248's pc doesn't even support right now) because testcontainers is not built to be used like this apparently and has a bunch of fucking bullshitty crap that it does ugh

In conclusion this has been a waste of time, i doubt this will ever get merged or leave draft state, and we're probably going to move back to mariadb-maven-plugin because i have no idea what i'm doing or why i'm on the solar mc team

@auriium auriium requested a review from A248 March 15, 2021 08:51
@auriium auriium added bug Something isn't working help wanted Extra attention is needed low priority labels Mar 15, 2021
@auriium
Copy link
Contributor Author

auriium commented Mar 15, 2021

Side note: I've tested it on my discord bot just now. It works fine asides from two points

1: Testcontainers annoying thread bs that i had to fix with the copy pasted crap from the thread that abuses reflection
(solution: docker rewrite never)

2: Modularization does not work, from what i've tested, it always generates the sources into the correct target in the right package, but when reaching the compile phase it complains that they don't exist when they legit do
(solution: no clue)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed low priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant