Skip to content
New issue

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

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

Already on GitHub? # to your account

Jakarta Tags 3.0 Spec #457

Merged
merged 7 commits into from
May 17, 2022
Merged

Jakarta Tags 3.0 Spec #457

merged 7 commits into from
May 17, 2022

Conversation

volosied
Copy link
Contributor

@volosied volosied commented Feb 24, 2022

Specification PR template

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

Include the following in the PR:

Note: If any item does not apply, check it and mark N/A below it.

@volosied volosied marked this pull request as draft February 24, 2022 16:06
@netlify
Copy link

netlify bot commented Feb 24, 2022

Deploy Preview for jakartaee-specifications ready!

Name Link
🔨 Latest commit 90f4809
🔍 Latest deploy log https://app.netlify.com/sites/jakartaee-specifications/deploys/627d7002085d9c00088a8859
😎 Deploy Preview https://deploy-preview-457--jakartaee-specifications.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@volosied volosied force-pushed the tags_3ga branch 2 times, most recently from 4b594f9 to d7689aa Compare February 24, 2022 16:52
@volosied volosied changed the title Jakarta Tags 3.0 Draft Jakarta Tags 3.0 Spec Feb 24, 2022
@ivargrimstad ivargrimstad added the release review Use this label on PRs that are filed for release review label Feb 27, 2022
@volosied
Copy link
Contributor Author

Reminder: jakartaee/platform-tck#882 (comment)

@starksm64
Copy link
Contributor

starksm64 commented Apr 6, 2022

Spec Review Checklist (Mentor)

  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
  • javadocs-jar artifact matches apidocs (optional for this release)
  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
  • Correct Logo
  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
  • SHA-256 fingerprint matches staged TCK zip file on cert request
  1. If a Release Review is required, the specification project team contacts the EMO to initiate the review by sending an email to emo@eclipse.org.
    (A Release Review is not required if the current release is a Service Release based on a previously successful Major or Minor
    release as indicated by a release record on the project's Releases page, e.g., the Jakarta Servlet releases page.)

  2. Update Jakarta EE API jar

  • Update the Jakarta EE API jar by submitting a PR to the jakartaee-api project that updates the version number of your API jar file.

@starksm64
Copy link
Contributor

@volosied the specification pdf and html are using DRAFT as their status. Can you update these to either have no status line or Final?

@volosied
Copy link
Contributor Author

@starksm64 Just updated them to final.

Fix index.md

Add back TCK digests

Update Jakarta Tags Spec to FINAL

Update to 3.0.0 api docs
@volosied
Copy link
Contributor Author

@starksm64

Staging repository link of the form https://jakarta.oss.sonatype.org/content/repositories/staging/jakarta/{spec}/jakarta.{spec}-api/x.y.z/

Here's the link: https://jakarta.oss.sonatype.org/content/repositories/staging/jakarta/servlet/jsp/jstl/jakarta.servlet.jsp.jstl-api/3.0.0/

Should this be check-marked? or is something incorrect?


The Jakarta Tags TCK is passing. How can I move forward with it? (Zip, test results, etc)

@volosied
Copy link
Contributor Author

Scott posted this -- jakartaee/platform-tck#886 (comment)

@starksm64
Copy link
Contributor

So now what is needed is a candidate TCK bundle and a CCR for a canidate GlassFish release. Let me ask @scottmarlow about that.

@scottmarlow
Copy link
Contributor

@arjantijms
Copy link
Contributor

CCR: jakartaee/tags#226

@volosied
Copy link
Contributor Author

@arjantijms @starksm64 What are the next steps for the CCR? Also, what about the other items in the checklist list?

  • Updated release record (can someone explain this more in detail?)
  • Generated IP Log (How do we do this?)
  • Email to PMC
  • Start release review by emailing EMO

Thanks!

@arjantijms
Copy link
Contributor

You can generate the IP log via the Eclipse project page. There's a link for it on the right side of the page. The preview looks like this: https://projects.eclipse.org/projects/ee4j.jstl/releases/3.0.0/iplog/preview

IP log: https://dev.eclipse.org/ipzilla/show_bug.cgi?id=24062

@arjantijms
Copy link
Contributor

@starksm64
Copy link
Contributor

starksm64 commented Apr 25, 2022

This looks good to me. Has the EMO contacts @arjantijms with their tracking issue? They usually do this when you file the IP log.

@volosied
Copy link
Contributor Author

@starksm64 @arjantijms Please let me know if I can do anything to help. I'm not sure what else specifically needs to be done / followed up on. Thanks again for all your help!

@starksm64
Copy link
Contributor

@volosied this looks ready to go to ballot to me, so I have asked @Emily-Jiang to start the ballot. That is the next step in the release process.

@volosied volosied marked this pull request as ready for review April 25, 2022 18:55
@ivargrimstad ivargrimstad added the ballot Delivered to the Specification Committee for ballot label Apr 26, 2022
@arjantijms
Copy link
Contributor

Release review tracking issues: https://gitlab.eclipse.org/eclipsefdn/emo-team/emo/-/issues/277

@@ -9,7 +9,7 @@ for manipulating XML documents, internationalization tags, and SQL tags. It also
existing custom tags with Jakarta Standard Tag Library tags.

* [Jakarta Standard Tag Library 3.0 Release Record](https://projects.eclipse.org/projects/ee4j.jstl/releases/3.0.0)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ivargrimstad Wanted to address your comments mentioned here: https://www.eclipse.org/lists/jakarta.ee-spec/msg02435.html

I should remove this Release Record above?

Comment on lines 23 to 31
The plan for the release is to clean up open issues:
* General specification document cleanup.
* Clean up issues found during the Jakarta EE9 review ballot process: update if possible the tagdocs to include EFSL, Update tlddoc generation if possible to jakarta.
* Clean up issues found during the Jakarta EE9 review ballot process
* Update tagdocs to include EFSL
* Update tlddoc generation to jakarta.
* Remove any deprecated methods from the API if they exist.
* Rename java.sun.com / xmlns.jcp.org taglib URIs to new jakarta.tags.\* URNs. [Issue 144](https://github.com/eclipse-ee4j/jstl-api/issues/144)
* This alone should require a major release.
* Other small clean up issues
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ivargrimstad
Should I update this section to list a Release Summary (and include list of changes) or should I just remove entire section?

Pages 3.1 and a few others, for example, don't include a Plan or other summary.

https://deploy-preview-435--jakartaee-specifications.netlify.app/specifications/pages/3.1/

@@ -48,7 +51,7 @@ The following TCK updates will be required:
* [Jakarta Standard Tag Library 3.0 Release Record](https://projects.eclipse.org/projects/ee4j.jstl/releases/3.0.0)

# Compatible Implementations
* [EE4J implementation of Jakarta Standard Tag Library 3.0.0](https://github.com/eclipse-ee4j/jstl-api/releases/download/3.0.0-IMPL-RELEASE/jakarta.servlet.jsp.jstl-3.0.0.jar) using vehicle [Eclipse Glassfish x.0.0-x](https://github.com/eclipse-ee4j/glassfish/releases/download/)
* [EE4J implementation of Jakarta Standard Tag Library 3.0.0](https://github.com/eclipse-ee4j/jstl-api/releases/tag/3.0.0-IMPL-RELEASE)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI: Updated this to say our latest impl (as that's what was used against the TCK)

Comment on lines 22 to 30
This release includes the following changes:
* Renamed Tag URIs to 'jakarta.tags.*'
* Updated Jakarta Tags to Java 11
* Added JPMS Module Info Class
* Removed dependency on XML Binding from the API pom
* Updated tagdocs to include EFSL
* Updated tlddoc generation to support jakarta.
* General specification document cleanup.
* Clean up issues found during the Jakarta EE9 review ballot process
* Update tagdocs to include EFSL
* Update tlddoc generation to jakarta.
* Remove any deprecated methods from the API if they exist.
* Rename java.sun.com / xmlns.jcp.org taglib URIs to new jakarta.tags.\* URNs. [Issue 144](https://github.com/eclipse-ee4j/jstl-api/issues/144)
* This alone should require a major release.
* Other small clean up issues

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ivargrimstad @Emily-Jiang Let me know how this looks per the comments found in the ballot. Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

@volosied volosied May 12, 2022

Choose a reason for hiding this comment

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

I added links to the changes labeled under Changed between Jakarta Standard Tag Library 2.0 and Jakarta Standard Tag Library 3.0

As for the other changes (last 3), there were multiple aspects to it, so I didn't bother adding links.

@volosied volosied force-pushed the tags_3ga branch 2 times, most recently from 35b524e to db1c4c9 Compare May 12, 2022 20:36
@Emily-Jiang Emily-Jiang added approved The ballot was approved by the Specification Committee and removed ballot Delivered to the Specification Committee for ballot labels May 17, 2022
@Emily-Jiang
Copy link
Contributor

Emily-Jiang commented May 17, 2022

On ballot completion, the specification committee mentor:

  • adds this final checklist to the main PR.
  • adds the approved label to the PRs, and sends out the Ballot Summary per this template to the public Jakarta EE Specification Committee email list
  • 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. Manually editing the jenkins Build Information will help identify the build (ie. Mail 2.0 or CDI 3.0).
  • merges the specification (and apidocs) PRs, ensuring the "date:" field in the _index.md file has an appropriate value to allow publishing.
  • updates the specification page with the ballot results. This is normally done via a separate PR that should be reviewed, approved, and merged.
  • notifies the EMO of the ballot results by email to emo@eclipse-foundation.org. Just forward the ballot summary note sent earlier to the public Spec Committee email list.
  • creates an issue in the specification project that includes the following checklist for the specification project team:
    <issue>

@Emily-Jiang Emily-Jiang merged commit 2df4bd0 into jakartaee:master May 17, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
approved The ballot was approved by the Specification Committee release review Use this label on PRs that are filed for release review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants