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.

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

Jakarta Interceptors 1.2 #52

Merged
merged 8 commits into from Sep 9, 2019
Merged

Conversation

arjantijms
Copy link
Contributor

@arjantijms arjantijms commented Jul 22, 2019

Signed-off-by: arjantijms arjan.tijms@gmail.com

Specification PR template

When creating a specification project release review, create two PRs with the content divided as follows.

Include the following in PR#1:

Signed-off-by: arjantijms <arjan.tijms@gmail.com>
@dblevins
Copy link
Contributor

Spec status is Draft, but should be Final or Final Release.

Signed-off-by: arjantijms <arjan.tijms@gmail.com>
@starksm64
Copy link
Contributor

starksm64 commented Aug 12, 2019

Spec Review Checklist

  1. Spec PR
  1. _index.md
  1. javadocs
  • Footer contains Eclipse copyright and link to license
  • ESFL license is included, usually as doc-files/speclicense.html
  • no META-INF directory in PR
  1. Spec PDF
  • Correct spec title
  • Version number of the form x.y, not x.y.z
  • Correct Eclipse copyright line
  • No DRAFT or SNAPSHOT
  1. Spec HTML
  • Same as PDF
  1. TCK zip file
  • README file (optional for this release)
  • EFTL license file, preferably named LICENSE.md
  • User's Guide (or equivalent documentation)
  • How to test the Compatible Implementation(s) listed in _index.md above with the TCK (may be in UG)
  1. TCK User's Guide (or equivalent documentation)
  • Software requirements listed
  • Installation and configuration described
  • How to run tests
  • Where to file challenges
  1. Compatibility certification request
  • Request follows template
  • SHA-256 fingerprint matches staged TCK zip file
  • Request issue has certification label.
  1. TCK results summary
  • Page is hosted by Compatible Implementation project
  • Includes all information from certification request
  • Summary includes number of tests passed, failed, errors

Added TCK link, but with placeholder question marks for now

Signed-off-by: arjantijms <arjan.tijms@gmail.com>
* Added spec index
* Added Eclipse copyright
* Added Link to platform CTS

* Updated version to two digits

Signed-off-by: arjantijms <arjan.tijms@gmail.com>
@kwsutter
Copy link
Contributor

Comments...

@kwsutter kwsutter self-requested a review August 15, 2019 20:25
Copy link
Contributor

@kwsutter kwsutter left a comment

Choose a reason for hiding this comment

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

A couple of changes. Main question is about whether you should just be piggybacking on the Glassfish TCK results and CR.

@arjantijms
Copy link
Contributor Author

The link for the TCK in the _index.md file should be of the form
https://download.eclipse.org/jakartaee//./

If you mean for Interceptors; that one doesn't exist. There is no Interceptors TCK :(

@arjantijms
Copy link
Contributor Author

If you are using the Glassfish CI and TCK for the testing, should you just be referencing the Glassfish CR instead of creating a CR for Interceptors? jakartaee/platform#99

That's a good question. Hmmm, I'm not sure what the answer is. @bshannon ? @dblevins ?

@bshannon
Copy link
Contributor

Yes, you should just point to the GlassFish CR and the platform TCK.

@arjantijms
Copy link
Contributor Author

@bshannon okay, thanks for the confirmation.

Signed-off-by: arjantijms <arjan.tijms@gmail.com>
Signed-off-by: arjantijms <arjan.tijms@gmail.com>
Signed-off-by: arjantijms <arjan.tijms@gmail.com>
@bshannon
Copy link
Contributor

@arjantijms This PR should refer to a certification request in the interceptor-api project.

@arjantijms
Copy link
Contributor Author

@arjantijms This PR should refer to a certification request in the interceptor-api project.

I updated it once again to the existing CR in the interceptor-api project, see jakartaee/interceptors#49

Signed-off-by: arjantijms <arjan.tijms@gmail.com>
Copy link
Contributor

@bshannon bshannon left a comment

Choose a reason for hiding this comment

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

Marking this final.

@bshannon bshannon added final Ready for Vote and removed draft Work in Progress labels Aug 23, 2019
@arjantijms arjantijms marked this pull request as ready for review August 23, 2019 21:44
@waynebeaton waynebeaton added the ballot Delivered to the Specification Committee for ballot label Aug 26, 2019
@waynebeaton waynebeaton added platform Run ballot with the Jakarta Platform ballot Delivered to the Specification Committee for ballot and removed ballot Delivered to the Specification Committee for ballot labels Aug 26, 2019
@waynebeaton waynebeaton added the approved The ballot was approved by the Specification Committee label Sep 9, 2019
@dblevins
Copy link
Contributor

dblevins commented Sep 9, 2019

  • On ballot completion
  • The designated specification committee representative adds this final checklist to the main PR.
  • The specification committee representative merges the specification PRs.
  • The designated specification committee representative calculates the staged EFTL TCK signature and promotes it to the committee download area
    using the https://ci.eclipse.org/jakartaee-spec-committee/job/promote-release/ job.
  • The specification project member who created the staging release promotes the specification api jars to maven central. An example release job script can be found here https://wiki.eclipse.org/MavenReleaseScript.
  • The EMO updates the specification page with the ballot results.
    This list goes on the committed spec index page.
  • The specification project team should go through the merged spec website page to verify all the links are valid.
  • The specification project team should approve the compatibility request.
  • The compatible implementation project/vendor MUST send an email tck@eclipse.org for approval of the compatible implementation for trademark usage.
  • The specification project team should merge any final release branch as appropriate for the branch management for the project.

@dblevins dblevins merged commit e8e88da into jakartaee:master Sep 9, 2019
@arjantijms
Copy link
Contributor Author

arjantijms commented Sep 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved The ballot was approved by the Specification Committee ballot Delivered to the Specification Committee for ballot final Ready for Vote platform Run ballot with the Jakarta Platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants