Skip to content

Updates#818

Merged
manovotn merged 4 commits intojakartaee:mainfrom
Ladicek:updates
May 24, 2024
Merged

Updates#818
manovotn merged 4 commits intojakartaee:mainfrom
Ladicek:updates

Conversation

@Ladicek
Copy link
Copy Markdown
Member

@Ladicek Ladicek commented May 21, 2024 •
edited
Loading

  • move from 5.0.1.Alpha1-SNAPSHOT to 5.0.0.Alpha1-SNAPSHOT
  • move the base Java level to 17
  • move a few forgotten references from 4.1 to 5.0
  • remove deprecated methods from BeanManager

@Ladicek Ladicek requested review from manovotn and starksm64 May 21, 2024 14:47
manovotn
manovotn previously approved these changes May 21, 2024
View reviewed changes
@manovotn
Copy link
Copy Markdown
Contributor

manovotn commented May 21, 2024

move from 5.0.1.Alpha1-SNAPSHOT to 5.0.0.Alpha2-SNAPSHOT

Shouldn't this be 5.0.0.Alpha1-SNAPSHOT?

@manovotn manovotn dismissed their stale review May 21, 2024 15:29

Too hasty from me

manovotn
manovotn requested changes May 21, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@manovotn manovotn left a comment

Choose a reason for hiding this comment

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

Since you are removing the BM methods, we should be able to also remove the EL dep from CDI API pom.xml and only keep it in the CDI EL API.

@Ladicek Ladicek force-pushed the updates branch 2 times, most recently from c82dcad to 29a4a2b Compare May 22, 2024 07:12
@Ladicek
Copy link
Copy Markdown
Member Author

Ladicek commented May 22, 2024

Shouldn't this be 5.0.0.Alpha1-SNAPSHOT?

That's what I also thought, but there have apparently been several attempts to release 5.0.0.Alpha1 already. None of them appeared in Maven Central, but the commits are there. Not sure what to do about this...?

@manovotn
Copy link
Copy Markdown
Contributor

manovotn commented May 22, 2024

Shouldn't this be 5.0.0.Alpha1-SNAPSHOT?

That's what I also thought, but there have apparently been several attempts to release 5.0.0.Alpha1 already. None of them appeared in Maven Central, but the commits are there. Not sure what to do about this...?

With no release in Central and no tag either, does it matter?

@Ladicek
Copy link
Copy Markdown
Member Author

Ladicek commented May 22, 2024

Good point, I didn't notice there's no tag. I'll move back to 5.0.0.Alpha1-SNAPSHOT.

@Ladicek
Copy link
Copy Markdown
Member Author

Ladicek commented May 22, 2024

Done.

@manovotn manovotn merged commit 778d0bf into jakartaee:main May 24, 2024
@Ladicek Ladicek deleted the updates branch May 27, 2024 07:10
@manovotn manovotn mentioned this pull request May 28, 2024
Closed
@Ladicek Ladicek added this to the CDI 5.0 milestone Oct 22, 2024
@manovotn manovotn mentioned this pull request Feb 25, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Azquelt Azquelt Azquelt approved these changes

@manovotn manovotn manovotn approved these changes

@starksm64 starksm64 Awaiting requested review from starksm64

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

CDI 5.0

Development

Successfully merging this pull request may close these issues.

3 participants

@Ladicek @manovotn @Azquelt