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

Represent Java BaseStream<T, S> as array of T in generated schema #1360

Merged
merged 1 commit into from
Mar 8, 2023
Merged

Represent Java BaseStream<T, S> as array of T in generated schema #1360

merged 1 commit into from
Mar 8, 2023

Conversation

MikeEdgar
Copy link
Member

Fixes #1359

Signed-off-by: Michael Edgar [email protected]

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

93.8% 93.8% Coverage
0.0% 0.0% Duplication

phillip-kruger
phillip-kruger previously approved these changes Jan 19, 2023
View reviewed changes
@MikeEdgar MikeEdgar added this to the 3.1.3 milestone Jan 20, 2023
@MikeEdgar MikeEdgar marked this pull request as draft January 20, 2023 11:41
@MikeEdgar
Copy link
Member Author

Change back to draft. This change should use BaseStream instead of Stream to capture the IntStream, LongStream, and DoubleStream cases.

@MikeEdgar MikeEdgar removed this from the 3.1.3 milestone Jan 29, 2023
@MikeEdgar MikeEdgar changed the title Represent Java Stream<T> as array of T in generated schema Represent Java BaseStream<T, S> as array of T in generated schema Mar 5, 2023
@MikeEdgar MikeEdgar marked this pull request as ready for review March 5, 2023 13:22
@MikeEdgar MikeEdgar added this to the 3.2.1 milestone Mar 5, 2023
@MikeEdgar MikeEdgar requested a review from phillip-kruger March 6, 2023 12:14
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

98.9% 98.9% Coverage
0.0% 0.0% Duplication

@MikeEdgar MikeEdgar merged commit d2948c9 into smallrye:main Mar 8, 2023
@MikeEdgar MikeEdgar deleted the issue-1359 branch March 8, 2023 01:16
@Azquelt Azquelt mentioned this pull request Feb 21, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phillip-kruger phillip-kruger phillip-kruger approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.3.0
Development

Successfully merging this pull request may close these issues.

Scan Stream<T> as array of T
2 participants
@MikeEdgar @phillip-kruger