testing merge strat - do not merge ns/add to committer list #6783
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Updated string formatting on non-f-strings. Found all cases of strings separated by white space on a single line and removed white space separation. EX: "hello " "world" -> "hello world". * add changelog entry
* Create functors to initialize event types with str-type member attributes. Before this change, the spec of various classes expected base_msg and msg params to be str's. This assumption did not always hold true. post_init hooks ensures the spec is obeyed. * Add new changelog. * Add msg type change functor to a few other events that could use it. Co-authored-by: Mila Page <versusfacit@users.noreply.github.com>
…y. (#6095) Co-authored-by: Mila Page <versusfacit@users.noreply.github.com>
* convert 027 cycle test * remove no-op expect_pass=False * remove postgres from test names
* Convert test to functional set. * Remove old statement tests from integration test set. * Nix whitespace Co-authored-by: Mila Page <versusfacit@users.noreply.github.com>
* switch out to use our action * point to main
* point to correct workflow * add inputs
* Initial structured logging changes * remove "this" from core/dbt/events/functions.py * CT-1047: Fix execution_time definitions to use float * CT-1047: Revert unintended checking of changes to functions.py * WIP * first pass to resolve circular deps * more circular dep resolution * remove a bunch of duplication * move message into log line * update comments * fix field that wen missing during rebase * remove double import * remove some comments and extra code * fix pre-commit * rework deprecations * WIP converting messages * WIP converting messages * remove stray comment * WIP more message conversion * WIP more message conversion * tweak the messages * convert last message * rename * remove warn_or_raise as never used * add fake calls to all new events * fix some tests * put back deprecation * restore deprecation fully * fix unit test * fix log levels * remove some skipped ids * fix macro log function * fix how messages are built to match expected outcome * fix expected test message * small fixes from reviews * fix conflict resolution in UI Co-authored-by: Gerda Shank <gerda@dbtlabs.com> Co-authored-by: Peter Allen Webb <peter.webb@dbtlabs.com>
* converting 023_exit_codes_tests * use packages fixture, clean up test names
…ker (#6180) * Bump python from 3.10.7-slim-bullseye to 3.11.0-slim-bullseye in /docker Bumps python from 3.10.7-slim-bullseye to 3.11.0-slim-bullseye. --- updated-dependencies: - dependency-name: python dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * Add automated changelog yaml from template for bot PR Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Github Build Bot <buildbot@fishtownanalytics.com>
* Updates lib to use new profile name functionality * Adds changie entry * Fixes formatting
* Combine various print result log events with different levels * Changie * more merge cleanup * Specify DynamicLevel for event classes that must specify level
* Remove root_path * Bump manifest schema to 8 * Update tests and compability utility for v8, root_path removal
…/core (#6144) * Update colorama requirement in /core Updates the requirements on [colorama](https://github.com/tartley/colorama) to permit the latest version. - [Release notes](https://github.com/tartley/colorama/releases) - [Changelog](https://github.com/tartley/colorama/blob/master/CHANGELOG.rst) - [Commits](tartley/colorama@0.3.9...0.4.6) --- updated-dependencies: - dependency-name: colorama dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> * Add automated changelog yaml from template for bot PR Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Github Build Bot <buildbot@fishtownanalytics.com>
* converting 044_test_run_operations
* init pr for dbt_debug test conversion * removal of old test * minor test format change * add new Base class and Test classes * reformatting test, new method for capsys and error messgae to check, todo fix badproject * refomatting tests, ready for review * checking yaml file, and small reformat * modifying since update wasn't working in ci/cd
* update gh action set-output variables * add changie file
* convert 038_caching_tests * Adapt for dbt-snowflake * PR feedback * Reformat
* Mv incremental on_schema_change tests to 'adapter zone' * Use type_string() * Cleanup
…dicates` (#6628) * add backwards compatibility and default argument * changie <3 * Update .changes/unreleased/Fixes-20230117-101342.yaml Co-authored-by: Jeremy Cohen <jeremy@dbtlabs.com>
* CT-1786: Port docs tets to pytest * Add generated CLI API docs * CT-1786: Comply with the new style requirements Co-authored-by: Github Build Bot <buildbot@fishtownanalytics.com>
* Improve error message for packages missing `dbt_project.yml` * Use black formatting * Update capitalization of expected error message
* Code cleanup and adding stderr to capture dbt * Debug with --log-format json now prints structured logs. * Add changelog. * Move logs into miscellaneous and add values to test. * nix whitespace and fix log levels * List will now do structured logging when log format set to json. * Add a quick None check. * Add a get guard to class check. * Better null checking * The boolean doesn't reflect the original logic but a try-catch does. * Address some code review comments and get us working again. * Simplify logic now that we have a namespace object for self.config.args. * Simplify logic for json log format checking. * Simplify code for allowing our GraphTest cases to pass while also hiding compile stats from dbt ls/list . * Simplify structured logging types. * Fix up boolean logic and simplify via De'Morgan. * Nix unneeded fixture. Co-authored-by: Mila Page <versusfacit@users.noreply.github.com>
* Short-circuit set operations for nice speed boost * Add changelog * Fix issue * Update .changes/unreleased/Under the Hood-20230122-215235.yaml Co-authored-by: Doug Beatty <44704949+dbeatty10@users.noreply.github.com> Co-authored-by: Doug Beatty <44704949+dbeatty10@users.noreply.github.com>
* CT-1718: Add Note and Formatting event types * CT-1718: Add changelog entry
* Include adapter_response in run_result in NodeFinished log event * Changie
* convert 044_init_tests
* CT-1857: Event cleanup * Add changelog entry.
* init commit for column_types test conversion * init start of test_column_types.py * pass tes macros into both tests * remove alt tests, remove old tests, push up working conversion * rename base class, move to adapter zone so adapters can use * typo fix
…ing (#6713) * Log information about vars_hash, normalize cli_vars before hashing * Changie * Add to test_events.py * Update core/dbt/events/types.py Co-authored-by: Doug Beatty <44704949+dbeatty10@users.noreply.github.com>
* change `exposure_content` to `source_content` * Adding changelog Co-authored-by: Leah Antkiewicz <leah.antkiewicz@fishtownanalytics.com>
* Convert test and make it a bit more pytest-onic * Ax old integration test. * Run black on test conversion * I didn't like how pytest was running the fixture so wrapped it into a closure. * Merge converted test into persist docs. Co-authored-by: Mila Page <versusfacit@users.noreply.github.com>
* Optimization to remove graph analysis bottleneck in large dags. * Add changelog entry.
* Test converted and reformatted for pytest. * Ax old versions of 052 test * Nix the 'os' import and black format * Change names of models to be more PEP like * cleanup code Co-authored-by: Mila Page <versusfacit@users.noreply.github.com>
…leases (#6602) * Add release workflows * Update nightly-release.yml * Set default `test_run` value to `true` * Update .bumpversion.cfg * Resolve review comment - Update workflow docs - Change workflow name - Set `test_run` default value to `true` * Update Slack secret * PyPI
|
Closed-- was experimenting with merge strats |
iknox-fa
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
toxto updatepip(fixespsycopg2-binary @ 2.9.5) ([BUGFIX] Forcetoxto updatepip(fixespsycopg2-binary @ 2.9.5) #6134)core/dbt/README.mdto match current (Updatecore/dbt/README.mdto match current #6371)git grep(Treat dense text blobs as binary forgit grep#6462)update_event_statusearlier + rename an event (Callupdate_event_statusearlier + rename an event #6572)tests/(Reformattests/#6622)on_schema_changetests -> "adapter zone" (mvon_schema_changetests -> "adapter zone" #6618)incremental_predicates(add backwards compatibility and default argument forincremental_predicates#6628)dbt_project.yml(Improve error message for packages missingdbt_project.yml#6685)exposure_contenttosource_content(changeexposure_contenttosource_content#6739)resolves #
Description
Checklist
changie newto create a changelog entry