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

[BUGFIX] Corrected typographical errors in two docstrings #7506

Conversation

alexsherstinsky
Copy link
Contributor

Please annotate your PR title to describe what the PR does, then give a brief bulleted description of your PR below. PR titles should begin with [BUGFIX], [FEATURE], [DOCS], or [MAINTENANCE]. If a new feature introduces breaking changes for the Great Expectations API or configuration files, please also add [BREAKING]. You can read about the tags in our contributor checklist.

Changes proposed in this pull request:

  • JIRA: GREAT-761/GREAT-1285/DX-392

After submitting your PR, CI checks will run and @cla-bot will check for your CLA signature.

For a PR with nontrivial changes, we review with both design-centric and code-centric lenses.

In a design review, we aim to ensure that the PR is consistent with our relationship to the open source community, with our software architecture and abstractions, and with our users' needs and expectations. That review often starts well before a PR, for example in github issues or slack, so please link to relevant conversations in notes below to help reviewers understand and approve your PR more quickly (e.g. closes #123).

Previous Design Review notes:

Definition of Done

Please delete options that are not relevant.

  • My code follows the Great Expectations style guide
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added unit tests where applicable and made sure that new and existing tests are passing.
  • I have run any local integration tests and made sure that nothing is broken.

Thank you for submitting!

@netlify
Copy link

netlify bot commented Mar 29, 2023

Deploy Preview for niobium-lead-7998 canceled.

Name Link
🔨 Latest commit 910fe59
🔍 Latest deploy log https://app.netlify.com/sites/niobium-lead-7998/deploys/6424876b1a367400088d669a

@alexsherstinsky alexsherstinsky marked this pull request as ready for review March 29, 2023 18:46
@alexsherstinsky alexsherstinsky enabled auto-merge (squash) March 29, 2023 18:46
@ghost
Copy link

ghost commented Mar 29, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@alexsherstinsky alexsherstinsky merged commit 7be0a1d into develop Mar 29, 2023
44 checks passed
@alexsherstinsky alexsherstinsky deleted the feature/GREAT-761/GREAT-1285/DX-392/alexsherstinsky/rule_based_profiler/data_assistant/contrib/capital_one_data_profiler_column_domain_builder-2023_03_27-248 branch March 29, 2023 19:56
Shinnnyshinshin pushed a commit that referenced this pull request Mar 29, 2023
…mitted_during_full_ci_run' into m/dx-372/engine_execute_warning_sqlalchemy_2

* origin/m/_/fix_warnings_that_are_only_emitted_during_full_ci_run:
  use optional imports, add note about pandas 2.0, general cleanup
  [MAINTENANCE] Pass `PandasDatasource` `batch_metadata` as `kwargs` to remove possibility of `None` on `DataAsset` model (#7503)
  [BUGFIX] Corrected typographical errors in two docstrings (#7506)
  [FEATURE] Introducing CapitalOne DataProfilerColumnDomainBuilder as well as multiple improvements to CapitalOne codebase and testability. (#7498)
  Lint
  Swallow sqlalchemy warning when using earlier pandas version
  [FEATURE] `BatchMetadata` for all fluent `DataAsset`s (#7392)
  Swallow sqlalchemy warning when using earlier pandas version
Shinnnyshinshin pushed a commit to richardohara/great_expectations that referenced this pull request Mar 31, 2023
* develop: (137 commits)
  [RELEASE] 0.16.4 (great-expectations#7525)
  [MAINTENANCE] Remove airflow2 min depdency test. (great-expectations#7524)
  [MAINTENANCE] Dedicated airflow 2.2.0 async test (great-expectations#7518)
  [MAINTENANCE] Use YAMLHandler in tests and docs (great-expectations#7507)
  [MAINTENANCE] Revert PR 7490 (great-expectations#7515)
  [BUGFIX] Typo in min versions install (great-expectations#7516)
  [DOCS] corrects typo in GCS setup guide (great-expectations#7514)
  [DOCS] Testing ADR (great-expectations#7495)
  [MAINTENANCE] Bump numpy from 1.21.0 to 1.22.0 in /docs_rtd (great-expectations#7509)
  [MAINTENANCE] Bump nbconvert from 5.6.1 to 6.5.1 in /docs_rtd (great-expectations#7508)
  [MAINTENANCE] Bump jupyter-core from 4.6.3 to 4.11.2 in /docs_rtd (great-expectations#7496)
  [MAINTENANCE] Bump certifi from 2020.6.20 to 2022.12.7 in /docs_rtd (great-expectations#7497)
  [MAINTENANCE] Bump gitpython from 3.1.7 to 3.1.30 in /docs_rtd (great-expectations#7494)
  [MAINTENANCE] Fix sqlalchemy warnings for pandas + sql fluent datasources (great-expectations#7504)
  [MAINTENANCE] Make `dataset_name` a parameter for Expectations tests or create name from `Expectation` name, which ensures only limited number of tables created. (great-expectations#7476)
  [MAINTENANCE] Pass `PandasDatasource` `batch_metadata` as `kwargs` to remove possibility of `None` on `DataAsset` model (great-expectations#7503)
  [BUGFIX] Corrected typographical errors in two docstrings (great-expectations#7506)
  [FEATURE] Introducing CapitalOne DataProfilerColumnDomainBuilder as well as multiple improvements to CapitalOne codebase and testability. (great-expectations#7498)
  [FEATURE] `BatchMetadata` for all fluent `DataAsset`s (great-expectations#7392)
  [MAINTENANCE] Connection.connect warning for SQLAlchemy 2.0 compatibility (great-expectations#7489)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants