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

Add dbt-synapse 1.7.0rc1 to the 1.7 pre-release bundle #128

Merged
merged 8 commits into from
Mar 2, 2024

Conversation

mikealfare
Copy link
Contributor

dbt-synapse has a 1.7 pre-release available. This adds the pre-release to the pre-release bundle. dbt-synapse has not published a final release yet.

@mikealfare mikealfare self-assigned this Feb 29, 2024
@mikealfare mikealfare linked an issue Feb 29, 2024 that may be closed by this pull request
@mikealfare
Copy link
Contributor Author

This appears to be happening somewhere:

ERROR: dbt-postgres from git+https://github.com/dbt-labs/dbt-core.git@main#egg=dbt-postgres&subdirectory=plugins/postgres (from -r /Users/runner/work/dbt-core-bundles/dbt-core-bundles/release_creation/bundle/requirements/v0.0.latest.requirements.txt (line 2)) does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.

This doesn't exist on main. Since this is the 1.7 bundle, this should probably be @1.7.latest instead of @main.

@mikealfare
Copy link
Contributor Author

We're hitting rate limit errors, which we've seen in the past. Once run did succeed though, suggesting this is a viable bundle. I'll rerun other versions (in particular py38) if they fail.

@mikealfare
Copy link
Contributor Author

py311 passed for ubuntu, rerunning for macos. Older versions of python are failing due to dependency conflicts involving hatching. It seems like there are a combination of dependencies that make this require py311, which is not consistent with our support (py38+).

@mikealfare mikealfare marked this pull request as draft February 29, 2024 16:34
@colin-rogers-dbt
Copy link
Contributor

@mikealfare added hatch and dbt-common/dbt-adapters to the dev requirements, looks like it's passing now

@colin-rogers-dbt colin-rogers-dbt marked this pull request as ready for review March 2, 2024 00:52
@colin-rogers-dbt colin-rogers-dbt merged commit d5fb42c into main Mar 2, 2024
7 of 8 checks passed
@colin-rogers-dbt colin-rogers-dbt deleted the add-dbt-synapse-1-7 branch March 2, 2024 01:07
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.

Add dbt-synapse to 1.7.latest
2 participants