Skip to content

Conversation

@lidavidm
Copy link
Contributor

What's Changed

Closes #32.

@lidavidm lidavidm requested a review from amoeba December 22, 2025 02:34
@lidavidm lidavidm marked this pull request as ready for review December 22, 2025 02:35
pr_standard:
name: "Check PR"
runs-on: ubuntu-latest
if: github.event_name == 'pull_request'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was breaking this workflow 🤦

amoeba added a commit to adbc-drivers/databricks that referenced this pull request Dec 23, 2025
Copy link
Contributor

@amoeba amoeba left a comment

Choose a reason for hiding this comment

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

LGTM. Tested on adbc-drivers/databricks#99.

version = "0.1.0"

[tasks]
make = "adbc-make run build DRIVER=<{driver}> VERBOSE=true"
Copy link
Contributor

Choose a reason for hiding this comment

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

is <{ ... }> valid?

Copy link
Contributor

Choose a reason for hiding this comment

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

Guess it is.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I changed the default syntax to avoid conflicting with GitHub's syntax.

@lidavidm lidavidm merged commit 2885ea2 into main Dec 23, 2025
3 checks passed
@lidavidm lidavidm deleted the dev branch December 23, 2025 03:21
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.

[Workflow] generate doesn't create pixi.toml

3 participants