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

use the same base classpath for empty packages as is used for normal packages #769

Merged
merged 1 commit into from
Jul 10, 2023

Conversation

bpholt
Copy link
Contributor

@bpholt bpholt commented Jun 21, 2023

References #768, #702, #743

Like @rossabaker in #743, I also don't know much about MiMa's internals, but from what I've poked around at, this seems to make sense?

@lightbend-cla-validator
Copy link
Collaborator

Hi @bpholt,

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

@bpholt
Copy link
Contributor Author

bpholt commented Jun 21, 2023

FYI, I signed the updated CLA

@bpholt
Copy link
Contributor Author

bpholt commented Jun 23, 2023

@SethTisue friendly ping to see if you can approve this workflow for CI

Copy link
Collaborator

@dwijnand dwijnand left a comment

Choose a reason for hiding this comment

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

Yeah, that makes more sense.

@SethTisue SethTisue self-assigned this Jul 3, 2023
@SethTisue SethTisue merged commit 7662b5a into lightbend-labs:main Jul 10, 2023
10 checks passed
@SethTisue
Copy link
Collaborator

Should I close #768 and/or #702?

@bpholt bpholt deleted the issue-768 branch July 18, 2023 17:59
@bpholt
Copy link
Contributor Author

bpholt commented Jul 18, 2023

I think we could close #768 at least, once this is released.

@SethTisue
Copy link
Collaborator

We close an issue when a fix is merged, even if it isn't released yet.

I'm closed #768, and I'm going to guess it was just an oversight that #702 wasn't closed when #743 was merged, so I closed that one too.

@SethTisue
Copy link
Collaborator

as for releasing, I've opened #774 and self-assigned

@bpholt
Copy link
Contributor Author

bpholt commented Jul 18, 2023

Works for me! Thank you!

mergify bot added a commit to slick/slick that referenced this pull request Jul 18, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [com.typesafe:sbt-mima-plugin](https://togithub.com/lightbend/mima) |
plugin | patch | `1.1.2` -> `1.1.3` |

---

### Release Notes

<details>
<summary>lightbend/mima (com.typesafe:sbt-mima-plugin)</summary>

### [`v1.1.3`](https://togithub.com/lightbend/mima/releases/tag/1.1.3)

[Compare
Source](https://togithub.com/lightbend/mima/compare/1.1.2...1.1.3)

#### Notable changes

- use the same base classpath for empty packages as is used for normal
packages by [@&#8203;bpholt](https://togithub.com/bpholt) in
[lightbend-labs/mima#769

#### Other changes

- build tweaks by [@&#8203;SethTisue](https://togithub.com/SethTisue) in
[lightbend-labs/mima#752
- update releasing steps by
[@&#8203;SethTisue](https://togithub.com/SethTisue) in
[lightbend-labs/mima#751
- Update coursier to 2.1.2 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#756
- Update coursier to 2.1.3 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#757
- Update coursier to 2.1.4 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#759
- Update coursier to 2.1.5 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#767
- Update munit to 1.0.0-M8 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#766
- Update nscplugin, sbt-scala-native, ... to 0.4.14 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#763
- Update sbt to 1.8.3 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#760
- Update sbt to 1.9.0 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#762
- Update sbt to 1.9.1 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#770
- Update sbt to 1.9.2 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#773
- Update sbt-dynver to 5.0.1 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#758
- Update sbt-mima-plugin to 1.1.2 by
[@&#8203;SethTisue](https://togithub.com/SethTisue) in
[lightbend-labs/mima#750
- Update sbt-scala-native-crossproject to 1.3.0 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#753
- Update sbt-scala-native-crossproject to 1.3.1 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#755
- Update sbt-scala-native-crossproject to 1.3.2 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#772
- Update scala-library, scala-reflect to 2.12.18 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#764
- Update scala-library, scala-reflect to 2.13.11 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#765
- Update scala3-library to 3.3.0 by
[@&#8203;scala-steward](https://togithub.com/scala-steward) in
[lightbend-labs/mima#761

#### New Contributors

- [@&#8203;bpholt](https://togithub.com/bpholt) made their first
contribution in
[lightbend-labs/mima#769

**Full Changelog**:
lightbend-labs/mima@1.1.2...1.1.3

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log [here](https://developer.mend.io/github/slick/slick).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi44LjExIiwidXBkYXRlZEluVmVyIjoiMzYuOC4xMSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->
This pull request was closed.
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