search

duckdb / dbt-duckdb

dbt (http://getdbt.com) adapter for DuckDB (http://duckdb.org)
Apache License 2.0
788 stars 70 forks source link

Only run MotherDuck integration tests on master, skip for pull requests #378

Closed guenp closed 2 months ago

guenp commented 2 months ago

This PR adds a job that checks if the MotherDuck token is defined, and only runs the integration tests if it is. This way the job is just skipped instead of failing with a Cannot connect to MotherDuck server: no token provided. error. Additionally, I've updated the push trigger branch name to master so the tests always run after PRs are merged.

For more detailed info see:

https://securitylab.github.com/research/github-actions-preventing-pwn-requests/ https://gist.github.com/GuillaumeFalourd/d3524cf6361afad87b9da3bf78270bc1

jwills commented 2 months ago

Thank you @guenp !