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

Python upgrade to 3.9 #113

Merged
merged 7 commits into from Feb 6, 2024
Merged

Python upgrade to 3.9 #113

merged 7 commits into from Feb 6, 2024

Conversation

rdeshmukh15
Copy link
Contributor

@rdeshmukh15 rdeshmukh15 commented Feb 2, 2024

Description of change

  • Python upgrade to 3.9.6 (TDL-24826)
  • requests upgrade to 2.31.0

Manual QA steps

  • Integration tests are passing locally as below :
INFO **************************************************************************************************************
2024-02-06 06:36:16,166 TapTester - INFO     sync mode exited succesfully
INFO sync mode exited succesfully
2024-02-06 06:36:16,199 TapTester - INFO     OK: tap_exit_status for sync job(3-49-1707201363) is 0
INFO OK: tap_exit_status for sync job(3-49-1707201363) is 0
2024-02-06 06:36:16,199 TapTester - INFO     OK: target_exit_status for job(3-49-1707201363) is 0
INFO OK: target_exit_status for job(3-49-1707201363) is 0
2024-02-06 06:36:16,199 TapTester - INFO     OK: discovery_exit_status for job(3-49-1707201363) is 0
INFO OK: discovery_exit_status for job(3-49-1707201363) is 0
2024-02-06 06:36:16,199 TapTester - INFO     OK: check_exit_status for job(3-49-1707201363) is None
INFO OK: check_exit_status for job(3-49-1707201363) is None
2024-02-06 06:36:16,208 TapTester - INFO     WARN: missing streams: {'items', 'bank_transfers', 'linked_transactions', 'receipts', 'credit_notes', 'invoices', 'overpayments', 'accounts', 'employees', 'manual_journals', 'payments', 'users', 'purchase_orders', 'contacts', 'bank_transactions', 'expense_claims', 'quotes', 'prepayments'}. consider adding them to the test account
INFO WARN: missing streams: {'items', 'bank_transfers', 'linked_transactions', 'receipts', 'credit_notes', 'invoices', 'overpayments', 'accounts', 'employees', 'manual_journals', 'payments', 'users', 'purchase_orders', 'contacts', 'bank_transactions', 'expense_claims', 'quotes', 'prepayments'}. consider adding them to the test account
2024-02-06 06:36:16,208 TapTester - INFO     Output records passing schema validation:
INFO Output records passing schema validation:
2024-02-06 06:36:16,208 TapTester - INFO        journals = 280  pks: ['JournalID']
INFO    journals = 280  pks: ['JournalID']
2024-02-06 06:36:16,209 TapTester - INFO        branding_themes = 3     pks: ['BrandingThemeID']
INFO    branding_themes = 3     pks: ['BrandingThemeID']
2024-02-06 06:36:16,209 TapTester - INFO        contact_groups = 2      pks: ['ContactGroupID']
INFO    contact_groups = 2      pks: ['ContactGroupID']
2024-02-06 06:36:16,209 TapTester - INFO        currencies = 1  pks: ['Code']
INFO    currencies = 1  pks: ['Code']
2024-02-06 06:36:16,209 TapTester - INFO        organisations = 1       pks: ['OrganisationID']
INFO    organisations = 1       pks: ['OrganisationID']
2024-02-06 06:36:16,209 TapTester - INFO        repeating_invoices = 6  pks: ['RepeatingInvoiceID']
INFO    repeating_invoices = 6  pks: ['RepeatingInvoiceID']
2024-02-06 06:36:16,209 TapTester - INFO        tax_rates = 7   pks: ['TaxType']
INFO    tax_rates = 7   pks: ['TaxType']
2024-02-06 06:36:16,210 TapTester - INFO        tracking_categories = 1 pks: ['TrackingCategoryID']
INFO    tracking_categories = 1 pks: ['TrackingCategoryID']
2024-02-06 06:36:16,210 TapTester - INFO     ***** Test Results *****
----------------------------------------------------------------------
Ran 3 tests in 93s

OK  
INFO ***** Test Results *****
----------------------------------------------------------------------
Ran 3 tests in 93s

OK 

Risks

Rollback steps

  • revert this branch

@rdeshmukh15 rdeshmukh15 force-pushed the python-upgrade-to-3.9 branch 2 times, most recently from 00a9148 to 15ff15a Compare February 6, 2024 07:13
@rdeshmukh15 rdeshmukh15 merged commit e2536cd into master Feb 6, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants