Skip to content

Commit

Permalink
Publish the wheels on PyPI
Browse files Browse the repository at this point in the history
  • Loading branch information
Twixes committed Oct 5, 2023
1 parent 3b70ecf commit 50a11d7
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 3 deletions.
18 changes: 16 additions & 2 deletions .github/workflows/build-hogql-parser.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,10 @@ on:
- .github/workflows/build-hogql-parser.yml

jobs:
build_wheels:
build-wheels:
name: Build wheels on ${{ matrix.os }}
runs-on: ${{ matrix.os }}
timeout-minutes: 15
timeout-minutes: 30
strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -44,3 +44,17 @@ jobs:
- uses: actions/upload-artifact@v3
with:
path: hogql_parser/wheelhouse/*.whl

upload-all:
needs: build-wheels
environment: pypi-hogql-parser
permissions:
id-token: write
runs-on: ubuntu-22.04
steps:
- uses: actions/download-artifact@v3
with:
name: artifact
path: dist

- uses: pypa/gh-action-pypi-publish@release/v1
2 changes: 1 addition & 1 deletion hogql_parser/parser.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,7 @@ class HogQLParseTreeConverter : public HogQLParserBaseVisitor {
);

PyObject* array_join_list = visitAsPyObject(array_join_clause_ctx->columnExprList());
for (size_t i = 0; i < PyList_Size(array_join_list); i++) {
for (Py_ssize_t i = 0; i < PyList_Size(array_join_list); i++) {
PyObject* expr = PyList_GET_ITEM(array_join_list, i);
if (!is_ast_node_instance(expr, "Alias")) {
Py_DECREF(array_join_list);
Expand Down

0 comments on commit 50a11d7

Please sign in to comment.