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

[BACKPORT] pin click #8051

Closed
wants to merge 66 commits into from
Closed

[BACKPORT] pin click #8051

wants to merge 66 commits into from

Conversation

emmyoop
Copy link
Member

@emmyoop emmyoop commented Jul 7, 2023

resolves #8050

Description

Pin click click>=7.0,<8.1.4

This backport differs from main to reduce risk of anyone depending on a version of click~=7 since it's a previous version

Checklist

aranke and others added 30 commits April 13, 2023 16:31
(cherry picked from commit ada8860)

Co-authored-by: Jeremy Cohen <[email protected]>
) (#7405)

safe version attribute access in _check_resource_uniqueness
(cherry picked from commit 57e9096)

Co-authored-by: Michelle Ark <[email protected]>
…est version has been modified (#7439) (#7460)

(cherry picked from commit 2739d5f)

Co-authored-by: Michelle Ark <[email protected]>
…eRefResolver (#7438) (#7461)

(cherry picked from commit 9874f9e)

Co-authored-by: Michelle Ark <[email protected]>
* Latest version should use un-suffixed alias

* Latest version can be in un-suffixed file

* FYI when unpinned ref to model with prerelease version

* [WIP] Nicer error if versioned ref to unversioned model

* Revert "Latest version should use un-suffixed alias"

This reverts commit 3616c52.

* Revert "[WIP] Nicer error if versioned ref to unversioned model"

This reverts commit c9ae4af.

* Define real event for UnpinnedRefNewVersionAvailable

* Update pp test for implicit unsuffixed defined_in

* Add changelog entry

* Fix unit test

* marky feedback

* Add test case for UnpinnedRefNewVersionAvailable event

(cherry picked from commit d53bb37)

Co-authored-by: Jeremy Cohen <[email protected]>
…7548)

* Back compat for previous retrurn type of 'collect_freshness'

* Test fixups

* PR feedback
(cherry picked from commit 19d6dab)

Co-authored-by: Doug Beatty <[email protected]>
* Pin protobuf to >=4.0.0

* Changie

(cherry picked from commit d34c511)

Co-authored-by: Gerda Shank <[email protected]>
test statically parsed two-argument ref

(cherry picked from commit 0891aef)

Co-authored-by: Michelle Ark <[email protected]>
)

(cherry picked from commit 40aca4b)

Co-authored-by: Kshitij Aranke <[email protected]>
#7605)

* CT 2510 Throw error for duplicate versioned and non versioned model names (#7577)

* Check for versioned/unversioned duplicates

* Add new exception DuplicateVersionedUnversionedError

* Changie

* Handle packages when finding versioned and unversioned duplicates

(cherry picked from commit 29f2cfc)

* Issue AmbiguousAlias error after DuplicateResourceName
* Exclude some profile fields from Jinja rendering when they are not valid Jinja. (#7630)

* CT-2583: Exclude some profile fields from Jinja rendering.

* CT-2583: Add functional test.

* CT-2583: Change approach to password jinja detection

* CT-2583: Extract string constant and add additional checks

* CT-2583: Improve unit test coverage

* CT-2583: Update changelog entry to reflect new approach
FishtownBuildBot and others added 20 commits June 8, 2023 16:30
…ypes (#7512) (#7784)

(cherry picked from commit 05b0ebb)

Co-authored-by: Michelle Ark <[email protected]>
* update adapters url (#7779)

* update adapters url

in response to [docs.getedbt.com pr 3465](https://github.com/dbt-labs/docs.getdbt.com/issues/3465), updating this error message to point to the correct URL, which was recently changed.

old URL: https://docs.getdbt.com/docs/supported-data-platforms#adapter-installation
new URL: https://docs.getdbt.com/docs/connect-adapters#install-using-the-cli

thank you @dbeatty10 for your 🦅 👀 !

* adding changie entry

* Update .changes/unreleased/Breaking Changes-20230612-161159.yaml

---------

Co-authored-by: Emily Rockman <[email protected]>
# Conflicts:
#	core/dbt/exceptions.py

* update error msg

---------

Co-authored-by: mirnawong1 <[email protected]>
(cherry picked from commit 83d163a)

Co-authored-by: Jeremy Cohen <[email protected]>
Co-authored-by: Michelle Ark <[email protected]>
* Changelog entry

* Fix: `dbt show --inline` with `private` models (#7838)

* Add functional test

* Check resource_type before DbtReferenceError

* Changelog entry
(cherry picked from commit 9c7e01d)

Co-authored-by: Jeremy Cohen <[email protected]>
Co-authored-by: Michelle Ark <[email protected]>
* cherry pick f767943

* Regenerate event proto types

---------

Co-authored-by: Jeremy Cohen <[email protected]>
…textVar (#7949) (#7981)

Cherry-picked from commit 2e7c968

* Add task contextvars to events/contextvars.py

* Use events.contextvars instead of task.contextvars

* Changie
* Remove pin of sqlparse to below 0.4.4

* Changie
* pin click

* changelog
# Conflicts:
#	core/setup.py
@emmyoop emmyoop requested review from a team as code owners July 7, 2023 16:33
@emmyoop emmyoop requested review from nathaniel-may, aranke and VersusFacit and removed request for a team July 7, 2023 16:33
@cla-bot cla-bot bot added the cla:yes label Jul 7, 2023
@emmyoop emmyoop closed this Jul 7, 2023
@emmyoop emmyoop deleted the backport/8050-1.5 branch July 7, 2023 17:38
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.

8 participants