Skip to content
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

Vendor release-prep from centralized dbt-release, replace tox testing with spark repo testing #997

Merged
merged 2 commits into from
Mar 2, 2024

Conversation

mikealfare
Copy link
Contributor

No description provided.

@mikealfare mikealfare self-assigned this Mar 2, 2024
@cla-bot cla-bot bot added the cla:yes label Mar 2, 2024
Copy link
Contributor

github-actions bot commented Mar 2, 2024

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the dbt-spark contributing guide.

- name: Run unit tests
run: python -m pytest --color=yes --csv unit_results.csv -v tests/unit

run-integration-tests:
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 is pulled from integration.yml. The steps that swap branches for dbt-core testing are removed to avoid adding inputs.

git commit -m "$commit_message"
git push

run-unit-tests:
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 is pulled from main.yml. The step that uploads the artifact is removed as it's not necessary.

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 file was vendored from dbt-labs/dbt-release/.github/workflows/release-prep.yml.

@mikealfare mikealfare merged commit d568d21 into main Mar 2, 2024
25 of 27 checks passed
@mikealfare mikealfare deleted the fix-release branch March 2, 2024 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants