Skip to content

fix(ci): verify executable step path#1

Merged
francoischalifour merged 4 commits into
mainfrom
fix/ci-verify-executable-path
Jun 26, 2026
Merged

fix(ci): verify executable step path#1
francoischalifour merged 4 commits into
mainfrom
fix/ci-verify-executable-path

Conversation

@francoischalifour

Copy link
Copy Markdown
Member

Summary

  • Fix the Verify executable CI step to use bin/altertable from the repo root instead of ../bin/altertable
  • The step has no working-directory: cli, so the previous path resolved outside the repository and caused chmod: cannot access '../bin/altertable': No such file or directory

Test plan

  • Smoke tested locally: bin/altertable --version and bin/altertable --help from repo root
  • CI Test TypeScript CLI job passes on this PR

The step runs from the repo root, not cli/, so ../bin/altertable did not exist on the runner.
Citty runs both the parent api handler and method subcommands, so api POST /path was also invoking POST /rest/v1/POST.
@francoischalifour francoischalifour changed the title Fix CI verify executable step path fix(ci): verify executable step path Jun 26, 2026
Cross-compiled altertable-linux-arm64 cannot execute on x64 ubuntu-latest hosts.
@francoischalifour francoischalifour merged commit 4044001 into main Jun 26, 2026
4 checks passed
francoischalifour added a commit that referenced this pull request Jul 9, 2026
🤖 I have created a release *beep* *boop*
---


## 1.0.0 (2026-07-09)


### Features

* add `altertable duckdb` command
([#27](#27))
([e114b11](e114b11))
* add `schema` command
([#21](#21))
([f713a80](f713a80))
* add first-class profile workflows and OAuth login profiles
([#23](#23))
([3ce655f](3ce655f))
* add one-shot shell completion installer
([#9](#9))
([3984fe8](3984fe8))
* auto-provision ephemeral lakehouse credentials after login
([#22](#22))
([d9d815f](d9d815f))
* avoid `--statement` flag for query
([#26](#26))
([23c5dc8](23c5dc8))
* **cli:** add origin-aware CLI update command
([#18](#18))
([5077028](5077028))
* **cli:** support lakehouse upsert endpoint
([#5](#5))
([d6ec3b8](d6ec3b8))
* implement `login` command
([#20](#20))
([1300577](1300577))
* improve human and agent experience
([#8](#8))
([135414f](135414f))
* refine shell completion UX
([#33](#33))
([bd2c605](bd2c605))
* replace `configure` by `profile --configure`
([#28](#28))
([fcf05b5](fcf05b5))


### Bug Fixes

* **api:** normalize api args to ensure the HTTP verb is optional
([#11](#11))
([78b32df](78b32df))
* **api:** prevent wide API tables from soft-wrapping
([#13](#13))
([56fbeb6](56fbeb6))
* **ci:** verify executable step path
([#1](#1))
([4044001](4044001))
* **cli:** remove active context from usage output
([#32](#32))
([9d2a4ad](9d2a4ad))
* **lakehouse:** allow append task without append row flags
([#19](#19))
([6ded957](6ded957))
* main concurrent merge issue
([#37](#37))
([335252c](335252c))
* tighten body validation and stream timeout handling
([#16](#16))
([4c4ce08](4c4ce08))
* validate lakehouse upload files before streaming the payload
([#15](#15))
([0e49b6e](0e49b6e))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: François Chalifour <francoischalifour@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants