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

Qute: add support to iterate long numbers #41679

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

DaHoC
Copy link

@DaHoC DaHoC commented Jul 4, 2024

Add support to not only iterate Integer, but also Long in Qute

This comment was marked as resolved.

@quarkus-bot quarkus-bot bot added the area/qute The template engine label Jul 4, 2024
Copy link
Contributor

@mkouba mkouba left a comment

Choose a reason for hiding this comment

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

@mkouba mkouba changed the title #6963 Add Qute support to iterate long numbers Qute: add support to iterate long numbers Jul 4, 2024
@geoand geoand changed the title Qute: add support to iterate long numbers Add support to iterate long numbers in Qute templates Jul 4, 2024
@mkouba mkouba changed the title Add support to iterate long numbers in Qute templates Qute: add support to iterate long numbers Jul 4, 2024
@mkouba
Copy link
Contributor

mkouba commented Jul 4, 2024

@DaHoC could you pls squash the commits; and do not include merge commits pls ;-).

@geoand

This comment was marked as duplicate.

@DaHoC DaHoC force-pushed the feature/6963_qute_iterate_long branch from 3b542d7 to bc8370b Compare July 4, 2024 08:13
@DaHoC DaHoC force-pushed the feature/6963_qute_iterate_long branch from bc8370b to e0cabae Compare July 4, 2024 08:13
Copy link

quarkus-bot bot commented Jul 4, 2024

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit e0cabae.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

⚠️ There are other workflow runs running, you probably need to wait for their status before merging.

@mkouba mkouba added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jul 4, 2024
Copy link

quarkus-bot bot commented Jul 4, 2024

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit e0cabae.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@geoand geoand merged commit 6c515be into quarkusio:main Jul 4, 2024
55 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.13 - main milestone Jul 4, 2024
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jul 4, 2024
@gastaldi
Copy link
Contributor

gastaldi commented Jul 4, 2024

@DaHoC your PR should have - Fixes #6963 in the description, but that's okay, thanks!

Copy link

github-actions bot commented Jul 4, 2024

🙈 The PR is closed and the preview is expired.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

4 participants