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

testing - do not merge #9821

Closed
wants to merge 25 commits into from
Closed

testing - do not merge #9821

wants to merge 25 commits into from

Conversation

emmyoop
Copy link
Member

@emmyoop emmyoop commented Mar 26, 2024

resolves #

Problem

Solution

Checklist

  • I have read the contributing guide and understand what's expected of me
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
  • This PR includes type annotations for new and modified functions

github-actions bot and others added 25 commits April 12, 2022 17:10
* Bumping version to 1.1.0rc1

* Adding changelog

* Updating adapter versions for Docker

* Remove extra spaces

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: Leah Antkiewicz <[email protected]>
Co-authored-by: leahwicz <[email protected]>
…) (#5126)

* Restore ability to configure and utilize `updated_at` for snapshots using the check_cols strategy

* Changelog entry

* Optional comparison of column names starting with `dbt_`

* Functional test for check cols snapshots using `updated_at`

* Comments to explain the test implementation

(cherry picked from commit d09459c)

Co-authored-by: Doug Beatty <[email protected]>
* Bumping version to 1.1.0rc2

* release changelog

* fix trailing space

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: Chenyu Li <[email protected]>
* fix retry logic failures

* changelog

* add tests to make sure data is getting where it needs to

* rename file

* remove duplicate file

(cherry picked from commit 31a3f2b)
* Even more scrubbing

* Changelog entry

* Even more

* remove reduendent scrub

* remove reduendent scrub

* fix encoding issue

* keep scrubbed log in args

Co-authored-by: Chenyu Li <[email protected]>
(cherry picked from commit ce0bcc0)

Co-authored-by: Jeremy Cohen <[email protected]>
…dering selectors (#5161)

* Use yaml renderer (with target context) for rendering selectors (#5136)

* Use yaml renderer (with target context) for rendering selectors

* Changie

* Convert cli_vars tests

* Add test for var in profiles

* Add test for cli vars in packages

* Add test for vars in selectors

(cherry picked from commit 1f898c8)

* Tweak cli_vars_in_packages test to do a run and build an adapter

Co-authored-by: Gerda Shank <[email protected]>
* Bumping version to 1.1.0rc3

* Update changelog

* Fixing spacing issue

* Removing more whitespace

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: Leah Antkiewicz <[email protected]>
Co-authored-by: leahwicz <[email protected]>
* Bumping version to 1.1.0

* Updating changelog

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: Leah Antkiewicz <[email protected]>
* Flexibilize MarkupSafe pinned version

The current `MarkupSafe` pinned version has been added in #4746 as a
temporary fix for #4745.

However, the current restrictive approach isn't compatible with other
libraries that could require an even older version of `MarkupSafe`, like
Airflow `2.2.2` [0], which requires `markupsafe>=1.1.1, <2.0`.

To avoid that issue, we can allow a greater range of supported
`MarkupSafe` versions. Considering the direct dependency `dbt-core` has
is `Jinja2==2.11.3`, we can use its pinning as the lower bound, which is
`MarkupSafe>=0.23` [1].

This fix should be also backported this to `1.0.latest` for inclusion in
the next v1.0 patch.

[0] https://github.com/adamantike/airflow/blob/2.2.2/setup.cfg#L125
[1] https://github.com/pallets/jinja/blob/2.11.3/setup.py#L53

Co-authored-by: Michael Manganiello <[email protected]>
* Bumping hologram version

* Add automated changelog yaml from template

* Updating issue

* Loosen requirement range

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

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: Jeremy Cohen <[email protected]>
(cherry picked from commit aa8115a)

Co-authored-by: leahwicz <[email protected]>
…5256)

* When parsing 'all_sources' should be a list of unique dirs

* Changie

* Fix some unit tests of all_source_paths

* Convert 039_config_tests

* Remove old 039_config_tests

* Add test for duplicate directories in 'all_source_files'

(cherry picked from commit f633e99)

Co-authored-by: Gerda Shank <[email protected]>
* Bumping version to 1.1.1rc1

* Remove whitespace

* Changelog update

* Fixing adapter versions

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: leahwicz <[email protected]>
Co-authored-by: Leah Antkiewicz <[email protected]>
…ting doc node (#5293)

* CT-159 Remove docs file from manifest when deleting doc node (#5270)

* Convert partial parsing docs tests

* Failing test

* Remove doc file from manifest when doc node is removed

* Changie

(cherry picked from commit cda88d1)

* Fix rm_file method

* comment out pip upgrade

Co-authored-by: Gerda Shank <[email protected]>
* Update context readme, small code cleanup (#5334)

* Pin networkx dependency to <2.8.4 for v1.1
* Fixing Windows color regression

* Cleaning up logic

* Consolidating logic to the logger

* Cleaning up vars

* Updating comment

* Removing unused import

* Fixing whitespace

* Adding changelog

(cherry picked from commit e48f7ab)

Co-authored-by: leahwicz <[email protected]>
* Bumping version to 1.1.1rc2

* Remove whitespace

* Add Changelog

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: Leah Antkiewicz <[email protected]>
* Bumping version to 1.1.1

* Fix whitespace

* Add Changelog

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: Leah Antkiewicz <[email protected]>
* Add functional test

* Add is_compatible_version logic

* Add changelog entry

* Fix mypy
* Backport #5382 to 1.1.latest

* Rm changelog entry for previous change
* Bumping version to 1.1.2rc1

* Fix whitespace

* Update changelog

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: leahwicz <[email protected]>
Co-authored-by: Leah Antkiewicz <[email protected]>
* Add changelog and whitespace fix to version bump Action

* Fixing whitespace

* Remove tabs

* Update .github/workflows/version-bump.yml

Co-authored-by: Emily Rockman <[email protected]>

* Update .github/workflows/version-bump.yml

Co-authored-by: Emily Rockman <[email protected]>

* Update .github/workflows/version-bump.yml

Co-authored-by: Emily Rockman <[email protected]>

* Update .github/workflows/version-bump.yml

Co-authored-by: Emily Rockman <[email protected]>

* Updating per comments

* Fix whitespace

Co-authored-by: Emily Rockman <[email protected]>

Co-authored-by: Emily Rockman <[email protected]>
* Bumping version to 1.1.2 and generate CHANGELOG

* Remove newline

Co-authored-by: Github Build Bot <[email protected]>
Co-authored-by: leahwicz <[email protected]>
@cla-bot cla-bot bot added the cla:yes label Mar 26, 2024
Copy link
Contributor

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 contributing guide.

@emmyoop emmyoop marked this pull request as ready for review March 26, 2024 00:27
@emmyoop emmyoop requested a review from a team as a code owner March 26, 2024 00:27
@emmyoop emmyoop closed this Mar 26, 2024
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.

4 participants