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

DockerPlugin code cleanup #1436

Merged
merged 15 commits into from
Sep 18, 2021
Merged

DockerPlugin code cleanup #1436

merged 15 commits into from
Sep 18, 2021

Conversation

frankivo
Copy link
Contributor

Some preparations to work on #1417

Copy link
Contributor

@muuki88 muuki88 left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the cleanup, especially in a separate PR ❤️ And thanks for your patience.

Only one small change then this is good to merge 😃

Co-authored-by: Nepomuk Seiler <muuki88@users.noreply.github.com>
@lightbend-cla-validator

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

Co-authored-by: Nepomuk Seiler <muuki88@users.noreply.github.com>
@lightbend-cla-validator

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

@frankivo
Copy link
Contributor Author

frankivo commented Jul 21, 2021

Thanks a lot for the cleanup, especially in a separate PR ❤️ And thanks for your patience.

Only one small change then this is good to merge 😃

Applied the changes :)

I'm not quite sure what the lightbend-cla-validator is on about.
Well, I guess I understand it. Just not sure how to resolve it, yet.

@frankivo
Copy link
Contributor Author

I feel like the best solution is to recreate those commits + pr?

@frankivo
Copy link
Contributor Author

I'll be traveling as of Thursday. If you can find a way to merge this, please do :)

@muuki88 muuki88 added the docker label Jul 29, 2021
@lightbend-cla-validator

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

@muuki88
Copy link
Contributor

muuki88 commented Jul 29, 2021

Sorry for the delay. Had a rough week ( lower back pain ) 😅

Can you run scalafmt for a correct formatting? Then this is good to go

@lightbend-cla-validator

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

@lightbend-cla-validator

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

@frankivo
Copy link
Contributor Author

@muuki88 ping

@lightbend-cla-validator

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

@muuki88
Copy link
Contributor

muuki88 commented Sep 14, 2021

There are two compilation errors after updating to master and the CLA must be signed 🤗

@frankivo
Copy link
Contributor Author

I'm confused, The error logs seems to be for scala 2.10 ?

@muuki88
Copy link
Contributor

muuki88 commented Sep 17, 2021

Sbt 0.13 is scala 2.10 and the plugin is still cross building

@lightbend-cla-validator

At least one pull request committer is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user

@muuki88 muuki88 merged commit 7ff7028 into sbt:master Sep 18, 2021
@frankivo
Copy link
Contributor Author

Holy smokes, it happened.

@frankivo frankivo deleted the cleanup branch September 18, 2021 20:48
@muuki88
Copy link
Contributor

muuki88 commented Sep 24, 2021

Thanks for your patience 💖

I try to act as fast as possible here, but family has higher priority 😊

@frankivo
Copy link
Contributor Author

No worries. Didn't mean to be salty :)

@muuki88
Copy link
Contributor

muuki88 commented Sep 26, 2021

Not at all 🤗 Thanks anyway for being so considerate ☺️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants