Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
72 commits
Select commit Hold shift + click to select a range
0a2f8bc
Experiment on removing ANTLR4
May 22, 2025
5b43788
Remove ANTLR for sql as well
May 23, 2025
19f1e80
Add ANTLR generation to task and exempt lint check for generated code
May 26, 2025
19a6cc4
Fix lint
May 26, 2025
8c8a6f3
Fix lint
May 26, 2025
8ba8191
Fix Devin comments
May 26, 2025
fc1f64c
Fix lint
May 26, 2025
cb707a7
Merge branch 'main' into xwei/-remove-antlr
anlowee May 27, 2025
a7072be
Test
May 27, 2025
17e3800
Add github workflow
May 27, 2025
6eda0ee
Merge branch 'xwei/-remove-antlr' of github.com:anlowee/clp into xwei…
May 27, 2025
bd0abcd
Fix
May 27, 2025
d0daacf
Fix
May 27, 2025
ddb047c
Fix
May 27, 2025
b9f16ea
Fix lint
May 27, 2025
b2ec42f
Fix
May 27, 2025
9d121af
Fix
May 27, 2025
1210f2b
Minor refactor
May 28, 2025
108f61d
Fix lint
May 28, 2025
f4742f2
Fix coderabbit
May 28, 2025
b8b804e
Merge branch 'main' into xwei/-remove-antlr
anlowee May 28, 2025
7878b3b
Merge branch 'main' into xwei/-remove-antlr
anlowee May 28, 2025
f457bff
Merge branch 'main' into xwei/-remove-antlr
anlowee May 29, 2025
09faf7c
Fix
May 30, 2025
075ca5b
Merge branch 'xwei/-remove-antlr' of github.com:anlowee/clp into xwei…
May 30, 2025
8b3191c
Merge branch 'main' into xwei/-remove-antlr
anlowee May 30, 2025
abcb91d
Merge branch 'main' into xwei/-remove-antlr
anlowee May 30, 2025
e88d914
Merge branch 'main' into xwei/-remove-antlr
anlowee May 30, 2025
da1523b
Add antlr-runtime task to download and install the antlr-runtime libr…
kirkrodrigues May 31, 2025
4f15581
Merge branch 'main' into xwei/-remove-antlr
anlowee Jun 2, 2025
7cec3fd
Merge branch 'main' into xwei/-remove-antlr
anlowee Jun 2, 2025
399803f
Merge branch 'main' into xwei/-remove-antlr
anlowee Jun 3, 2025
b7a0519
Merge branch 'main' into xwei/-remove-antlr
anlowee Jun 3, 2025
7507fec
Limit task concurrency to the number of available cores.
kirkrodrigues Jun 3, 2025
4bb484e
Test hard-coding number of parallel tasks that can be run.
kirkrodrigues Jun 3, 2025
cfd48c0
Execute one task at a time.
kirkrodrigues Jun 4, 2025
4a8ad65
Limit the antlr-runtime build to 2 cores.
kirkrodrigues Jun 4, 2025
0c3e5e1
Revert "Limit the antlr-runtime build to 2 cores."
kirkrodrigues Jun 4, 2025
5fee637
Revert "Execute one task at a time."
kirkrodrigues Jun 4, 2025
10baa2d
Revert "Test hard-coding number of parallel tasks that can be run."
kirkrodrigues Jun 4, 2025
51bcf30
Revert "Limit task concurrency to the number of available cores."
kirkrodrigues Jun 4, 2025
6760985
Fix
Jun 4, 2025
457254e
Test
Jun 4, 2025
e30dc2b
Test over
Jun 4, 2025
7b27d63
Fix all comments
Jun 4, 2025
0270ef2
Fix lint
Jun 4, 2025
c8277a0
Fix
Jun 4, 2025
4b83336
Fix
Jun 4, 2025
2980015
Fix some coderabbit comments
Jun 4, 2025
3ef0261
Merge branch 'main' into xwei/-remove-antlr
anlowee Jun 5, 2025
7727eed
Limit task concurrency to number of cores.
kirkrodrigues Jun 5, 2025
b5b2909
Don't use a shell variable.
kirkrodrigues Jun 5, 2025
6da4782
Limit task concurrency to 1.
kirkrodrigues Jun 5, 2025
e9c6c36
Account for shell variable expansion.
kirkrodrigues Jun 5, 2025
9d9d91c
Specify JOBS for antlr-runtime task.
kirkrodrigues Jun 5, 2025
c493f54
Revert "Account for shell variable expansion."
kirkrodrigues Jun 5, 2025
9a30c0b
Revert "Limit task concurrency to 1."
kirkrodrigues Jun 5, 2025
3bcf6fe
Revert "Don't use a shell variable."
kirkrodrigues Jun 5, 2025
6b4f74e
Revert "Limit task concurrency to number of cores."
kirkrodrigues Jun 5, 2025
616bb69
Merge branch 'main' into xwei/-remove-antlr
anlowee Jun 5, 2025
529787e
Fix
Jun 5, 2025
0f5cda6
Merge branch 'main' into xwei/-remove-antlr
anlowee Jun 5, 2025
660e2c5
Merge branch 'xwei/-remove-antlr' of github.com:anlowee/clp into xwei…
Jun 5, 2025
b86cc0a
Fix
Jun 5, 2025
bcbd0fe
Change to generated in lint.yaml
Jun 5, 2025
54b73cc
Fix
Jun 5, 2025
76fd933
Fix
Jun 5, 2025
37dbe46
Fix
Jun 6, 2025
b07a57a
Fix
Jun 6, 2025
9c741d0
Fix
Jun 6, 2025
8a9449e
Fix
Jun 6, 2025
b35a0c9
Fix
Jun 6, 2025
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
42 changes: 42 additions & 0 deletions .github/workflows/clp-s-generated-code-checks.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
name: "clp-s-generated-code-checks"

on:
pull_request:
push:
workflow_dispatch:

concurrency:
group: "${{github.workflow}}-${{github.ref}}"
# Cancel in-progress jobs for efficiency
cancel-in-progress: true

Comment on lines +8 to +12

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick (assertive)

Sanitise concurrency group
Using github.ref may include slashes (refs/heads/...), which can cause unexpected grouping. Consider using github.head_ref for PRs or replacing / with _ to ensure valid group IDs.

🤖 Prompt for AI Agents
In .github/workflows/clp-s-generated-code-checks.yaml around lines 8 to 12, the
concurrency group uses github.ref which can contain slashes, causing invalid
group IDs. Modify the concurrency group to sanitize github.ref by replacing
slashes with underscores or use github.head_ref for pull requests to ensure the
group ID is valid and consistent.

jobs:
antlr-code-committed:
name: "antlr-code-committed"
strategy:
matrix:
os: ["macos-latest", "ubuntu-latest"]
runs-on: "${{matrix.os}}"
steps:
- uses: "actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683"
with:
submodules: "recursive"

- name: "Install task"
shell: "bash"
run: "npm install -g @go-task/cli"

Comment on lines +25 to +28

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick (assertive)

Cache Task CLI install
Installing @go-task/cli on every run can slow CI. You could cache the npm global folder or use GitHub’s tool cache for faster workflows.

🤖 Prompt for AI Agents
In .github/workflows/clp-s-generated-code-checks.yaml around lines 25 to 28, the
workflow installs @go-task/cli globally on every run, which slows down the CI
process. Modify the workflow to cache the npm global folder or use GitHub
Actions' built-in tool cache to store the installed @go-task/cli between runs,
so the installation step can be skipped if the cache is valid, improving
workflow speed.

- if: "matrix.os == 'macos-latest'"
name: "Install coreutils (for md5sum)"
run: "brew install coreutils"

- name: "Check if the generated parsers are the latest"
shell: "bash"
run:
|
git status --porcelain \
components/core/src/clp_s/search/kql/generated \
components/core/src/clp_s/search/sql/generated \
| grep . > /dev/null \
&& exit 1 \
|| exit 0
9 changes: 3 additions & 6 deletions components/core/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -119,12 +119,9 @@ if (absl_FOUND)
endif()

# Find and setup ANTLR Library
# We build and link to the static library
find_package(ANTLR REQUIRED)
if (ANTLR_FOUND)
message(STATUS "Found ANTLR ${ANTLR_VERSION}")
else()
message(FATAL_ERROR "Could not find libraries for ANTLR ${ANTLR4_TAG}")
find_package(antlr4-runtime REQUIRED)
if (antlr4-runtime_FOUND)
message(STATUS "Found antlr4-runtime ${antlr4-runtime_VERSION}")
endif()
Comment on lines +123 to 125

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick (assertive)

Optional: simplify version message
Since find_package(... REQUIRED) already errors out if not found, you can remove the surrounding if guard and unconditionally call message(STATUS "Found antlr4-runtime ${antlr4-runtime_VERSION}") to streamline the code.

🤖 Prompt for AI Agents
In components/core/CMakeLists.txt around lines 123 to 125, remove the if
condition checking for antlr4-runtime_FOUND and unconditionally call
message(STATUS "Found antlr4-runtime ${antlr4-runtime_VERSION}"). This
simplifies the code because find_package with REQUIRED already ensures the
package is found or errors out.


# Find and setup Boost Library
Expand Down
180 changes: 0 additions & 180 deletions components/core/cmake/Modules/ExternalAntlr4Cpp.cmake

This file was deleted.

138 changes: 0 additions & 138 deletions components/core/cmake/Modules/FindANTLR.cmake

This file was deleted.

Loading