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.

Sign up for GitHub

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

Jump to bottom

Add Jakarta JSON Processing standalone TCK runner #24939

Merged
merged 4 commits into from
Apr 29, 2024
Merged

Add Jakarta JSON Processing standalone TCK runner #24939

merged 4 commits into from
Apr 29, 2024

Conversation

avpinchuk
Copy link
Contributor

To run TCK:

mvn clean install -Ptck -pl :glassfish-external-tck-jsonp

Results:

[INFO] 
[INFO] Results:
[INFO] 
[INFO] Tests run: 179, Failures: 0, Errors: 0, Skipped: 0
[INFO]

@avpinchuk
Add Jakarta JSON Processing standalone TCK runner
a1cd09e 
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
@avpinchuk avpinchuk added this to the 7.0.15 milestone Apr 27, 2024
@avpinchuk avpinchuk self-assigned this Apr 27, 2024
@pzygielo

This comment was marked as resolved.

Sign in to view
@avpinchuk
Copy link
Contributor Author

avpinchuk commented Apr 27, 2024 •
edited
Loading

Test report generation failed on JDK 18+.

@avpinchuk
Disable test report generation on JDK 18+
7c3963b 
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
pzygielo
pzygielo previously approved these changes Apr 28, 2024
View reviewed changes
@pzygielo pzygielo dismissed their stale review April 28, 2024 15:34

Need more time to understand how this works

@avpinchuk avpinchuk marked this pull request as draft April 28, 2024 16:17
@avpinchuk
Fixes Maven warnings
60df857 
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
@avpinchuk
Update SigTest Maven Plugin to version 2.2
6295a0d 
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
@avpinchuk avpinchuk marked this pull request as ready for review April 28, 2024 20:00
@arjantijms arjantijms merged commit 6a77e1c into eclipse-ee4j:master Apr 29, 2024
2 checks passed
@avpinchuk avpinchuk deleted the jsonp-tck branch May 1, 2024 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arjantijms arjantijms arjantijms approved these changes

@dmatej dmatej Awaiting requested review from dmatej

@hs536 hs536 Awaiting requested review from hs536

@pzygielo pzygielo Awaiting requested review from pzygielo

Assignees

@avpinchuk avpinchuk

Labels
build and test improvement
Projects
None yet
Milestone
7.0.15
Development

Successfully merging this pull request may close these issues.
3 participants
@avpinchuk @pzygielo @arjantijms