-
Notifications
You must be signed in to change notification settings - Fork 77
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.
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
Missing copyright headers on the new BCE #597
Comments
And why is that a problem? I was under the impression that we only need the license in the repo? |
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. |
Perhaps https://www.eclipse.org/projects/handbook/#ip-copyright-headers is applicable
|
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
|
This has been added via #598 |
Signed-off-by: Scott M Stark <[email protected]>
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.
The text was updated successfully, but these errors were encountered: