eclipse-theia / theia-cloud

Eclipse Public License 2.0
61 stars 34 forks source link

Publish maven artifacts to GH Packages #305

Closed sgraband closed 5 months ago

sgraband commented 5 months ago

Provide reusable maven ci workflow. Publish maven-conf and org.eclipse.theia.cloud.common. Publish org.eclipse.theia.cloud.operator and org.eclipse.theia.cloud.service. Always resolve org.eclipse.theia.cloud.common locally. Update actions/setup-java@v3 to actions/setup-java@v4.

Successful publish runs (hardcoded to run on PR): maven-conf common operator service

Some observations/questions:

  1. We could probably remove the service tests CI as this is now indirectly als called on the maven build. WDYT?
  2. The naming on the checks (which are starting to become quite a few) could be improved imho. Maybe in a follow up?
sgraband commented 5 months ago

Thanks for the feedback. I applied your feedback and removed the service tests. I also created a reminder to discuss the renaming of the ci jobs.