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

Handle POM-only modules by creating empty Definitions #743

Merged
merged 1 commit into from
Mar 27, 2023

Conversation

rossabaker
Copy link
Contributor

References #702.

I don't know much about MiMa's internals. This may be nonsense, but it fixes typelevel/otel4s#122.

@lightbend-cla-validator
Copy link
Collaborator

Hi @rossabaker,

Thank you for your contribution! We really value the time you've taken to put this together.

We see that you have signed the Lightbend Contributors License Agreement before, however, the CLA has changed since you last signed it.
Please review the new CLA and sign it before we proceed with reviewing this pull request:

https://www.lightbend.com/contribute/cla

@SethTisue SethTisue self-assigned this Feb 11, 2023
@rossabaker
Copy link
Contributor Author

NoPackageInfo was alluring, but throws an exception, and the sealed trait precludes introducing alternate dummy definitions. This is the "emptiest" one I could come up with.

@SethTisue
Copy link
Collaborator

SethTisue commented Mar 27, 2023

Optimistically merging 🤷

Thank you!

@SethTisue SethTisue merged commit 3f4903a into lightbend-labs:main Mar 27, 2023
@SethTisue
Copy link
Collaborator

Would having this in a release save you any time/trouble? We haven't released here in 6 months, but I looked and this PR is the only PR merged in that time that isn't just a Steward version bump.

@armanbilge
Copy link
Contributor

It certainly would, a number of builds have applied a workaround for this including otel4s, typelevel-scalafix, and typelevel/toolkit off the top of my head.

@SethTisue SethTisue mentioned this pull request Mar 27, 2023
19 tasks
@SethTisue
Copy link
Collaborator

okay, I've made #749 and assigned myself; I should be able to find the time to take care of it within the next few weeks

LuciferYang pushed a commit to apache/spark that referenced this pull request May 24, 2023
### What changes were proposed in this pull request?
The pr aims to upgrade mima-core from 1.1.0 to 1.1.2.

### Why are the changes needed?
- New version includes bug fixed, eg:
1.Handle POM-only modules by creating empty Definitions by rossabaker in lightbend-labs/mima#743

- Release note:
1.https://github.com/lightbend/mima/releases/tag/1.1.2
2.https://github.com/lightbend/mima/releases/tag/1.1.1

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Pass GA.

Closes #41294 from panbingkun/SPARK-43771.

Authored-by: panbingkun <[email protected]>
Signed-off-by: yangjie01 <[email protected]>
czxm pushed a commit to czxm/spark that referenced this pull request Jun 12, 2023
### What changes were proposed in this pull request?
The pr aims to upgrade mima-core from 1.1.0 to 1.1.2.

### Why are the changes needed?
- New version includes bug fixed, eg:
1.Handle POM-only modules by creating empty Definitions by rossabaker in lightbend-labs/mima#743

- Release note:
1.https://github.com/lightbend/mima/releases/tag/1.1.2
2.https://github.com/lightbend/mima/releases/tag/1.1.1

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Pass GA.

Closes apache#41294 from panbingkun/SPARK-43771.

Authored-by: panbingkun <[email protected]>
Signed-off-by: yangjie01 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants