-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate CircleCI to GitHub Actions #162
Conversation
* Add support for unit tests * Update dependencies to match `dbt-snowflake` * Update tox config to match `dbt-snowflake` * Update setup.py to match `dbt-snowflake`
* Unlike other DBT adapters, this one contains support for three different flavors of MySQL, thus has a handful of duplicate files. Therefore W002 should be ignored, as duplicates are not an error.
* Used `dbt-snowflake` as an example
Integration tests won't run on this PR because the I did verify that these tests run and pass in an earlier commit where I temporarily specified the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks awesome @mwallace582 🚀
Thanks for your work on this.
# git+https://github.com/dbt-labs/[email protected]#egg=dbt-core&subdirectory=core | ||
git+https://github.com/dbt-labs/[email protected]#egg=dbt-tests-adapter&subdirectory=tests/adapter |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note for future: depending on the version number of the next release of dbt-mysql, we'll need to change 1.4.latest
to match that version.
resolves #120
Description
Migrate Circle CI lint, unit, and integration tests to GitHub Actions
dbt-snowflake
anddbt-bigquery
were used as inspiration for these changes, and I've updated this repo in many places to more closely follow those examples.Checklist
CHANGELOG.md
with information about my change