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

[MAINTENANCE] Don't include 'test-pipeline' in extras_require dict #5659

Merged
merged 1 commit into from
Aug 3, 2022

Conversation

kenwade4
Copy link
Contributor

@kenwade4 kenwade4 commented Aug 3, 2022

A new requirements-dev-*.txt file was introduced in #5646

There's no reason to have "test-pipeline" as an extras_require key (i.e. pip install "great_expectations[test-pipeline]"), so it is removed

Changes proposed in this pull request:

  • Don't include 'test-pipeline' in extras_require dict

@github-actions github-actions bot added the devrel This item is being addressed by the Developer Relations Team label Aug 3, 2022
@ghost
Copy link

ghost commented Aug 3, 2022

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

@netlify
Copy link

netlify bot commented Aug 3, 2022

Deploy Preview for niobium-lead-7998 ready!

Name Link
🔨 Latest commit 603c9d2
🔍 Latest deploy log https://app.netlify.com/sites/niobium-lead-7998/deploys/62ea6091c1c73f000838a164
😎 Deploy Preview https://deploy-preview-5659--niobium-lead-7998.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@kenwade4 kenwade4 enabled auto-merge (squash) August 3, 2022 11:51
@kenwade4 kenwade4 merged commit cf55939 into develop Aug 3, 2022
@kenwade4 kenwade4 deleted the misc/setup-fix branch August 3, 2022 11:55
Shinnnyshinshin pushed a commit that referenced this pull request Aug 5, 2022
…etric-change

* develop: (30 commits)
  Add mysql client to Dockerfile.tests (#5681)
  [DOCS]added guide how to use great expectations with aws glue (#5536)
  [MAINTENANCE] Assign `resource_type` in `InlineStoreBackend` constructor (#5671)
  [RELEASE] 0.15.17 (#5675)
  fix: patch issue with 'id' vs 'id_' (#5677)
  Tidying up  the texts and config in gen; bump ops (#5678)
  [MAINTENANCE] data_context initial type checking (#5662)
  [FEATURE]Enable showing expectation suite by domain and by expectation_type -- from DataAssistantResult (#5673)
  [MAINTENANCE] Ensure that `id_` fields in Marshmallow schema serialize as `id` (#5660)
  [MAINTENANCE] Improve DateTime Conversion Candling in Comparison Metrics & Expectations and Provide a Clean Object Model for Metrics Computation Bundling (#5656)
  [MAINTENANCE] Only run Cloud E2E tests in primary pipeline
  Refactor docker test CI steps into jobs. (#5665)
  [MAINTENANCE] move tool config to pyproject.toml (#5649)
  add expectation for crc32 (#5580)
  Quick bug fix to all profile numeric column bounds expectations (#5651)
  Don't include 'test-pipeline' in extras_require dict (#5659)
  [ANALYTICS] add new invoke tasks to `tasks.py` and create new file `usage_stats_utils.py` (#5593)
  [MAINTENANCE] Introduce running tests in docker on our Azure CI pipeline. (#5646)
  [MAINTENANCE] Add E2E Cloud test for `DataContext.add_checkpoint()` (#5653)
  Make comparisons of aggregate values date aware (#5642)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devrel This item is being addressed by the Developer Relations Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants