Missing copyright headers on the new BCE

[Emily-Jiang]

The new classes under https://github.com/eclipse-ee4j/cdi/tree/master/api/src/main/java/jakarta/enterprise/inject/build/compatible/spi have no copyright header.

[manovotn]

And why is that a problem? I was under the impression that we only need the license in the repo?
Lots of classes have the jboss header still which, I believe, is now not relevant either. What kind of header should be here?

[Emily-Jiang]

I will find out whether it is mandatory to have the copyright headers as I have not seen Jakarta EE API classes without any copyright headers.

[pzygielo]

Perhaps https://www.eclipse.org/projects/handbook/#ip-copyright-headers is applicable

Where possible, all source code [...] should contain appropriate copyright

[Emily-Jiang]

Thank you @pzygielo for the link! I talked with @ivargrimstad and he confirmed that the new classes need to have copyright headers. I guess the following copyright template might work. More details can be found from the link provided by @pzygielo

/********************************************************************************

• Copyright (c) {year} {owner}[ and others]
• This program and the accompanying materials are made available under the Apache Software License 2.0
• which is available at https://www.apache.org/licenses/LICENSE-2.0.
• SPDX-License-Identifier: Apache-2.0
  ********************************************************************************/

[starksm64]

This has been added via #598