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.

Sign up for GitHub

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

Jump to bottom

Remove Salesforce integration tests from CI #709

Merged
merged 2 commits into from
Oct 26, 2020
Merged

Remove Salesforce integration tests from CI #709

merged 2 commits into from
Oct 26, 2020

Conversation

cgardens
Copy link
Contributor

@cgardens cgardens commented Oct 26, 2020 •
edited
Loading

  • The Salesforce API call quota is such that we can't run these integration tests on every PR build.
  • Each discover call, costs about 0.5% of the 24-hour API quota. In these tests that gets called at least 4 times in each run of the standard test suite so that's about 2% of the quota per build. Our options right now are to push less code or remove these tests for salesforce from the CI. 😉

@jrhizor
Copy link
Contributor

jrhizor commented Oct 26, 2020

Maybe we could run it on master only starting tomorrow?

@cgardens cgardens merged commit d3f7afc into master Oct 26, 2020
@cgardens cgardens deleted the cgardens/remove_salesforce_integration_tests_from_ci branch October 26, 2020 21:27
davydov-d added a commit that referenced this pull request Oct 20, 2022
@davydov-d
#709 source stripe: upd stream schema for payment_intents stream
7e785b0
davydov-d added a commit that referenced this pull request Oct 20, 2022
@davydov-d
#709 source stripe: upd changelog
b6ee212
davydov-d added a commit that referenced this pull request Oct 21, 2022
@davydov-d
Merge branch 'master' into ddavydov/#709-source-stripe-upd-stream-schema
7dc9400
davydov-d added a commit that referenced this pull request Oct 21, 2022
@davydov-d @octavia-squidington-iii
Source Stripe: update stream schema for payment_intents stream (#18228
067e36d 
)

* #709 source stripe: upd stream schema for payment_intents stream

* #709 source stripe: upd changelog

* auto-bump connector version

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
jhammarstedt pushed a commit to jhammarstedt/airbyte that referenced this pull request Oct 31, 2022
@davydov-d @octavia-squidington-iii @jhammarstedt
Source Stripe: update stream schema for payment_intents stream (air…
3ea57a4 
…bytehq#18228)

* airbytehq#709 source stripe: upd stream schema for payment_intents stream

* airbytehq#709 source stripe: upd changelog

* auto-bump connector version

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sherifnada sherifnada sherifnada approved these changes

@jrhizor jrhizor jrhizor approved these changes

@ChristopheDuong ChristopheDuong Awaiting requested review from ChristopheDuong

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cgardens @jrhizor @sherifnada