Skip to content
Snippets Groups Projects

feat: Upgrade Quarkus to 2.14.2 + integrate api-common library

3 unresolved threads

Resolves #190 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
1 1 org.eclipsefoundation.adopters.api.ProjectsAPI/mp-rest/url=https://projects.eclipse.org
2 2
3 3 ## OAUTH CONFIG
4 quarkus.http.port=8080
  • If you change this line, we have to also update the k8s mappings under src/main/k8s to change the service from port 8080 to 8090. Could you add that update to this PR?

  • Please register or sign in to reply
  • 12 <maven.compiler.source>11</maven.compiler.source>
    13 <maven.compiler.target>11</maven.compiler.target>
    14 <maven.compiler.parameters>true</maven.compiler.parameters>
    15 <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
    16 <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
    17 <quarkus.platform.artifact-id>quarkus-bom</quarkus.platform.artifact-id>
    18 <quarkus.platform.group-id>io.quarkus.platform</quarkus.platform.group-id>
    19 <quarkus.platform.version>2.11.2.Final</quarkus.platform.version>
    20 <surefire-plugin.version>3.0.0-M5</surefire-plugin.version>
    21 <sonar.sources>src/main</sonar.sources>
    22 <sonar.tests>src/test</sonar.tests>
    23 <sonar.java.coveragePlugin>jacoco</sonar.java.coveragePlugin>
    24 <sonar.dynamicAnalysis>reuseReports</sonar.dynamicAnalysis>
    25 <sonar.jacoco.reportPaths>${project.build.directory}/jacoco-report</sonar.jacoco.reportPaths>
    26 <sonar.junit.reportPath>${project.build.directory}/surefire-reports</sonar.junit.reportPath>
    27 <eclipse-api-version>0.7.0</eclipse-api-version>
  • Additionally, the Adopters service uses Jsonb, where we should change to Jackson ObjectMapper to be consistent with our other projects

  • I can't put a comment on a line I saw, which is weird. But we should remove JSONB from the POM, as we use Jackson now.

        <dependency>
          <groupId>io.quarkus</groupId>
          <artifactId>quarkus-resteasy-jsonb</artifactId>
        </dependency>
  • :white_check_mark: Deploy Preview for adopters-www-preview.eclipse.org is ready [:stopwatch: 1 min 31 sec]

    :mag: Inspect the build logs

    :whale: Test locally docker run -it --rm -p 18181:8080 docker.io/eclipsefdn/adopters-www-preview-eclipse-org:mr-199--d8be15d-b5 and open to http://localhost:18181

    :rocket: Browse the preview: https://preview-199--adopters-www-preview-eclipse-org.eclipsecontent.org/

    Edited by Web Dev Bot user
  • added 1 commit

    • e6e39d8a - feat: Migrate models to AutoValue + migrate to jackson

    Compare with previous version

  • :white_check_mark: Deploy Preview for adopters-www-preview.eclipse.org is ready [:stopwatch: 2 min 4 sec]

    :mag: Inspect the build logs

    :whale: Test locally docker run -it --rm -p 18181:8080 docker.io/eclipsefdn/adopters-www-preview-eclipse-org:mr-199--801aac9-b6 and open to http://localhost:18181

    :rocket: Browse the preview: https://preview-199--adopters-www-preview-eclipse-org.eclipsecontent.org/

    Edited by Web Dev Bot user
  • Martin Lowe
  • 79 80 <groupId>com.google.guava</groupId>
    80 81 <artifactId>guava</artifactId>
    81 82 </dependency>
    83
    84 <!--
    85 Annotation preprocessors - reduce all of the boiler plate -->
    86 <dependency>
  • :white_check_mark: Deploy Preview for adopters-www-preview.eclipse.org is ready [:stopwatch: 1 min 43 sec]

    :mag: Inspect the build logs

    :whale: Test locally docker run -it --rm -p 18181:8080 docker.io/eclipsefdn/adopters-www-preview-eclipse-org:mr-199--d6e9f5c-b7 and open to http://localhost:18181

    :rocket: Browse the preview: https://preview-199--adopters-www-preview-eclipse-org.eclipsecontent.org/

    Edited by Web Dev Bot user
  • Martin Lowe mentioned in commit 302fcef6

    mentioned in commit 302fcef6

  • merged

  • Please register or sign in to reply
    Loading