Skip to content

CAP Java: date/time functions #2527

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

Open
agoerler wants to merge 9 commits into
base: main
Choose a base branch
from
Open

CAP Java: date/time functions #2527

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
f1d2117
Java date/time functions
agoerler Apr 22, 2026
55cfa54
Merge remote-tracking branch 'origin/main' into javaFunctions
agoerler Apr 22, 2026
95d8c59
Update java/working-with-cql/query-api.md
agoerler Apr 22, 2026
b105dfc
cosmetics
smahati Apr 28, 2026
0ad32f9
Merge branch 'main' into javaDateTime
smahati Apr 28, 2026
d7f4e84
update link
smahati Apr 28, 2026
a8d2915
rename duplicate header
smahati Apr 28, 2026
f7b1787
Apply suggestions from code review
renejeglinsky Apr 29, 2026
cd7ca80
add check as expression to snippet checker
smahati May 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions .github/java-snippet-checker/check-java-snippets.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ for (const dir of baseDirs) {
const variations = [
{ content: snippet.content, error: null },
{ content: snippetAsMethod(snippet.content), error: null },
{ content: snippetAsExpression(snippet.content), error: null },
{ content: snippetAsCode(snippet.content), error: null },
];

Expand Down Expand Up @@ -206,6 +207,17 @@ ${ indentLines(content.trim(), 4) }
`;
}

/**
* @param {string} content
*/
function snippetAsExpression(content) {
Copy link
Copy Markdown
Contributor

@smahati smahati May 5, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

@agoerler I added a expression option based on the other variations, the idea is to allow expressions to get checked and it still catches issues in cases like an assignment with a missing semicolon so I hope this is ok but if you think this'll cause issues we can revert it and use a ignore comment instead.

return `// Snippet Checker
Object o =
${ indentLines(content.trim(), 4) }
;
`;
}

/**
* @param {string[]} files
*/
Expand Down
Loading
Loading