Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The initial migration of the CDI component specification EE integration requirements to the platform #838

Closed
wants to merge 37 commits into from
Closed

The initial migration of the CDI component specification EE integration requirements to the platform #838

wants to merge 37 commits into from

Conversation

starksm64
Copy link
Contributor

@starksm64 starksm64 commented Feb 1, 2024 •
edited

This is a work in progress to incorporate the CDI specification EE integration requirements to the platform.

This PR appears to include several commits in order to preserve the history of the related content from the CDI specification repository.

One big issue to resolve with this is how to handle the cross references to the CDI specification that no longer resolve. (FIXED)

fixes #837

antoinesd and others added 30 commits June 30, 2015 10:55
@antoinesd
Simplifying spec generation
0415b5c 
switch to Asciidcotor pdf generation (remove docbook).
Twick licence blocks to have a nice pdf version
@antoinesd
CDI-538 Section 3.8 on CDI 1.2 spec javax.security.Principal (now in …
e4a8eee 
…17.8)
@antoinesd
CDI-416 Typo on @PersistencContext
a74bf63
@jharting @antoinesd
Fix minor grammar mistakes
32f8856
@antoinesd
Introduce new ref in EE for TCK
44d071f
@tremes @antoinesd
Fix interceptors_ee.asciidoc file name.
9d84c2a
@antoinesd
Cleaning source asciidoc files to have one sentence per line
486a579
@antoinesd
Migrating to Asciidoctor 1.5+ notation
96082a9
@tremes @antoinesd
CDI-553 move notion of "security context" to EE part.
8445ba9
@tremes
CDI-545 observers can be only local bussines method.
d47e5eb
@antoinesd
CDI-569 Add @ObservesAsync where the spec mention @observes
c0efa85
@antoinesd
Correct sources to match one sentence per line
39f2808
@tremes @antoinesd
CDI-571 producer and disposer methods can be only local bussines meth…
3a4f2dc 
…od of EJB. (jakartaee#284)
@antoinesd
Correcting typos on Asciidoctor links
6cec83e
@tremes @antoinesd
CDI-555 remove the outdated wording linked to previous Java SE boot api
2f1f1f0
@mkouba @antoinesd
CDI-500 Clarify @Intercepted bean metadata injection for EE components (
a9c7fca 
jakartaee#318)

additional fix - missing chapter id. (jakartaee#325)
@mkouba @antoinesd
CDI-625 Make it clear when exactly are context init/destroy events fi…
c9387f7 
…red (jakartaee#303)

- add @BeforeDestroyed
@tremes @antoinesd
Few language updates (jakartaee#329)
d18cd84 
Nice to see text reviewer ;). Thx @tremes.
@tremes @antoinesd
CDI-667 introduce new chapter for trimmed bean archive in Java EE. (j…
5a79325 
…akartaee#368)
@antoinesd
Cleaning Javadoc and typos
066c3fe 
remove unused import
minor syntax update to java 8
@antoinesd
CDI-495 What happens if an illegal bean type is found in the set of b…
3bcfaf5 
…ean types (jakartaee#363)
@antoinesd
CDI-689 Typo or unclear message in 24.1.2 (jakartaee#382)
87f355b
@johnament @antoinesd
CDI-690 Request Context Clarification (jakartaee#385)
016781b 
Clarify in core when request context is active, and change verbiage a bit to align to common term of context.
@scottmarlow @starksm64
change from javax.* to jakarta.* (jakartaee#414)
1e0e34e 
Signed-off-by: Scott Marlow <smarlow@redhat.com>
@ivargrimstad
Jakartify (jakartaee#444)
3a885ba 
* Jakartify definition

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify decorators

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify events

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify implementation

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify inheritance

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify inject

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify interceptors

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify intro

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify lifecycle

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify packaging

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify scopes

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify spi

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify core packaging

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify core spi

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify inject

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Jakartify resolution

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Fix javax.* references to jakarta.*

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>

* Fix javax.annotation.ManagedBean reference in javadoc to jakarta.annotation.ManagedBean

Signed-off-by: Ivar Grimstad <ivar.grimstad@eclipse-foundation.org>
@starksm64
Fix the javax.servlet package usage
ab4339b 
Signed-off-by: Scott M Stark <starksm64@gmail.com>
@manovotn
Removal of deprecated APIs: BeanManager.fireEvent()
853406e
@manovotn
Removal of deprecated APIs: @New
76184af
@Ladicek @manovotn
comprehensive review and various Lite/Full split changes
7f667a2
@Ladicek
Rename files and chapter refereces to align with rest of the specific…
923ea63 
…ation
Ladicek and others added 5 commits May 24, 2023 09:46
@Ladicek
Bump CDI version to 4.1 and place EL integration API in a new supplem…
00fd2ff 
…ental artifact (jakartaee#644)

* Fix remnants of BeanManager.fireEvent() removal

* Centralize common Maven properties in the parent POM

* Fix EL import version in bundle metadata

* Bump CDI version to 4.1

* Place the EL integration API to ELAwareBeanManager in a new supplemental API artifact

The existing EL integration API in `BeanManager` is deprecated for removal.
@starksm64 @manovotn
Remove references to the removed jakarta.annotation.ManagedBean jakar…
0272260 
…taee#750

Signed-off-by: Scott M Stark <starksm64@gmail.com>
@starksm64
Move the CDI EE integration spec requirements to a new cdi-ee-spec di…
9990154 
…rectory

Signed-off-by: Scott M Stark <starksm64@gmail.com>
@starksm64
Merge cdi spec EE integration section into cdi-ee
a6ba5ce
@starksm64
The initial migration of the CDI component specification EE integrati…
20a8e09 
…on requirements, jakartaee#837

Signed-off-by: Scott M Stark <starksm64@gmail.com>
@starksm64 starksm64 mentioned this pull request Feb 1, 2024
Closed
3 tasks
@starksm64
Add an approach to correct the missing cross-references.
f53b005 
Signed-off-by: Scott M Stark <starksm64@gmail.com>
@starksm64
Add an approach to correct the missing cross-references.
fad1580 
Signed-off-by: Scott M Stark <starksm64@gmail.com>
@starksm64
Copy link
Contributor Author

The missing cross-references have been fixed by adding a xrefs.adoc section that points to the online CDI html specification sections.

@Emily-Jiang
Copy link
Contributor

@starksm64 how can we deal with the ECA check failures?

@starksm64
Copy link
Contributor Author

starksm64 commented Feb 16, 2024 •
edited

@starksm64 how can we deal with the ECA check failures?

We can't because by pulling in the history the check is looking at CDI committers who are not platform committers. We know where this came from so we will just have to ignore the failures. There are also people who are historic contributors before Jakarta existed and had to be exempted when the content was migrated into Jakarta.

@manovotn manovotn mentioned this pull request Feb 20, 2024
Closed
@starksm64 starksm64 mentioned this pull request Feb 22, 2024
Closed
2 tasks
@edburns edburns added the jea-linked Linked in jakarta-ee-azdo project label Mar 4, 2024
@edburns edburns mentioned this pull request Mar 4, 2024
Merged
@edburns
Copy link
Contributor

edburns commented Mar 5, 2024

I have taken actions to rebase this PR onto HEAD of main in PR #855. I recommend closing this PR and treating #855 as subsuming this PR.

@starksm64 starksm64 closed this Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
jea-linked Linked in jakarta-ee-azdo project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Migrate the CDI integration requirements from the CDI component project to the Platform project