Skip to content

Conversation

@romainreuillon
Copy link
Contributor

A first version of sbt-osgi cross compling to sbt 2.0.

I did the strict minimum to make the plugin and the test compile and run, so it is mergeable.

There is more to do especially on the caching side. I'll work on it when I'll have some time.

@romainreuillon
Copy link
Contributor Author

I guess don't exactly know how the testing part works, can somebody help on this part?

@eed3si9n
Copy link
Member

eed3si9n commented Sep 2, 2025

Could you run githubWorkflowGenerate to regenerate GitHub Action YAML files plz?

@romainreuillon
Copy link
Contributor Author

Of course, done

build.sbt Outdated

ThisBuild / (pluginCrossBuild / sbtVersion) := {
scalaBinaryVersion.value match {
case "2.12" => "1.5.8"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
case "2.12" => "1.5.8"
case "2.12" => "1.9.9"

If this plugin wants to run tests on JDK 21, then the minimum version should be 1.9.9.

@eed3si9n
Copy link
Member

eed3si9n commented Sep 2, 2025

https://github.com/sbt/sbt-osgi/blob/main/.scalafmt.conf probably needs to be updated to

runner.dialect = Scala212Source3

@romainreuillon
Copy link
Contributor Author

Thanks, that's a lot better. Some test are not passing yet. For what I understand it seems to be caused by some sbt 2 instabilities.

Is that what you get from the error as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants