Skip to content
This repository has been archived by the owner on Apr 10, 2024. It is now read-only.

feat: support for quarkus using CDI #91

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

feat: support for quarkus using CDI #91

wants to merge 8 commits into from

Conversation

csviri
Copy link
Collaborator

@csviri csviri commented Apr 17, 2023

No description provided.

@csviri csviri self-assigned this Apr 17, 2023
@csviri csviri marked this pull request as draft April 17, 2023 13:50
@csviri csviri requested a review from metacosm April 17, 2023 13:50
@csviri csviri marked this pull request as ready for review April 18, 2023 08:42
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
<properties>
<quarkus.platform.artifact-id>quarkus-bom</quarkus.platform.artifact-id>
<quarkus.platform.group-id>io.quarkus.platform</quarkus.platform.group-id>
<quarkus.platform.version>2.16.6.Final</quarkus.platform.version>
Copy link
Contributor

Choose a reason for hiding this comment

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

We should probably target Quarkus 3 (which is already tagged).

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

AFAIK there is no officeial release yet: https://github.com/quarkusio/quarkus/releases
neother on the website, also the automatic update script updates it into CR2

Copy link
Contributor

Choose a reason for hiding this comment

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

Final is already tagged so this is what should be used moving forward, imo.

@sonarcloud
Copy link

sonarcloud bot commented Apr 19, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

100.0% 100.0% Coverage
0.0% 0.0% Duplication

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants