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 Annotations 1.3 spec #69

Merged
merged 5 commits into from Aug 28, 2019
Merged

Jakarta Annotations 1.3 spec #69

merged 5 commits into from Aug 28, 2019

Conversation

m0mus
Copy link
Contributor

@m0mus m0mus commented Aug 1, 2019

Signed-off-by: Dmitry Kornilov dmitry.kornilov@oracle.com

Signed-off-by: Dmitry Kornilov <dmitry.kornilov@oracle.com>
Signed-off-by: Dmitry Kornilov <dmitry.kornilov@oracle.com>
@kwsutter kwsutter added the draft Work in Progress label Aug 5, 2019
@starksm64
Copy link
Contributor

It looks like there are supurious files in here:

  • annotations/1.3/license-efsl.adoc
  • annotations/1.3/images/jakarta_ee_logo_schooner_color_stacked_default.png

Copy link
Contributor

@starksm64 starksm64 left a comment

Choose a reason for hiding this comment

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

Are we strict about spec title including a version or not? These have "Jakarta Annotations 1.3" as the "Specification: " field value.

annotations/1.3/_index.md Outdated Show resolved Hide resolved
@starksm64
Copy link
Contributor

For wave 1, this spec should be due by 8/7, so please review this ASAP. Note there is a good checklist for https://github.com/jakartaee/specification-committee/blob/master/spec_review_checklist.md

Signed-off-by: Dmitry Kornilov <dmitry.kornilov@oracle.com>
Signed-off-by: Dmitry Kornilov <dmitry.kornilov@oracle.com>
@m0mus
Copy link
Contributor Author

m0mus commented Aug 9, 2019

It looks like there are supurious files in here:

* annotations/1.3/license-efsl.adoc

* annotations/1.3/images/jakarta_ee_logo_schooner_color_stacked_default.png

Fixed

@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
    ada069841408ba88378bb1813977ccdef4eb85e02097ba36de44bbd39cbd7c6c
  • 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

Copy link
Contributor

@starksm64 starksm64 left a comment

Choose a reason for hiding this comment

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

Looks complete except for the "Draft..." in the summary of the 1.3/_index.md

annotations/1.3/_index.md Outdated Show resolved Hide resolved
@starksm64 starksm64 added final Ready for Vote and removed draft Work in Progress labels Aug 12, 2019
@waynebeaton waynebeaton added the ballot Delivered to the Specification Committee for ballot label Aug 13, 2019
Signed-off-by: Dmitry Kornilov <dmitry.kornilov@oracle.com>
@waynebeaton waynebeaton added the approved The ballot was approved by the Specification Committee label Aug 28, 2019
@bshannon bshannon merged commit 34dc2ab into jakartaee:master Aug 28, 2019
@bshannon
Copy link
Contributor

bshannon commented Aug 28, 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.
  • 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 and send an email to tck@eclipse.org for approval of the compatible implementation for trademark usage.
  • The specification project team should merge any final release branch as approriate for the branch management for the project.

@dblevins
Copy link
Contributor

@m0mus The staged API binaries will be expired and deleted automatically by Nexus OSSRH on Sunday, Sept 1 st. Can you publish this api jar to Maven Central at your earliest convenience?

@m0mus
Copy link
Contributor Author

m0mus commented Aug 30, 2019

I don't see that it's published on the web site: https://jakarta.ee/specifications/

@m0mus m0mus deleted the ca_spec branch August 30, 2019 09:15
@m0mus
Copy link
Contributor Author

m0mus commented Aug 30, 2019

@dblevins Yes, we will release it today.

@m0mus
Copy link
Contributor Author

m0mus commented Sep 5, 2019

  • The specification project member who created the staging release promotes the specification api jars to maven central.
  • 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 and send an email to tck@eclipse.org for approval of the compatible implementation for trademark usage.
  • The specification project team should merge any final release branch as approriate for the branch management for the project.

I checked the links on the staged web site here. Everything is fine except a link to TCK. The link looks correct. I guess that it doesn't work because TCK is not available there yet. Please confirm.

@starksm64
Copy link
Contributor

I see the TCK under this directory:
https://download.eclipse.org/jakartaee/annotations/1.3/?d

@m0mus
Copy link
Contributor Author

m0mus commented Sep 5, 2019

@starksm64 How I can change the link on the web site?

@edbratt
Copy link
Contributor

edbratt commented Sep 5, 2019 via email

@m0mus
Copy link
Contributor Author

m0mus commented Sep 6, 2019

PR with TCK link fix submitted: #110

@bshannon
Copy link
Contributor

bshannon commented Sep 6, 2019

Looks like this spec is done!

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants