Skip to content

555 upgrade requests to the latest version#1075

Merged
EGrunzke merged 2 commits intomasterfrom
555-upgrade-requests-to-the-latest-version
Aug 4, 2025
Merged

555 upgrade requests to the latest version#1075
EGrunzke merged 2 commits intomasterfrom
555-upgrade-requests-to-the-latest-version

Conversation

@EGrunzke
Copy link
Contributor

@EGrunzke EGrunzke commented Jul 31, 2025

  • Updated requests to 2.32.4
  • Added .cache to .gitignore - pip puts a lot of files in there

EGrunzke added 2 commits July 31, 2025 10:06
Upgrade requests to 2.32.4 (required second run of pip-compile-all.sh to get requirements-test.txt in sync)
@EGrunzke EGrunzke requested a review from neob91-close July 31, 2025 16:58
@neob91-close
Copy link
Contributor

@EGrunzke asking just in case (because it's not mentioned in the PR description) - is this tested?

@EGrunzke
Copy link
Contributor Author

EGrunzke commented Aug 4, 2025

@EGrunzke asking just in case (because it's not mentioned in the PR description) - is this tested?

I ran the test_data script successfully. That sets up 6 emails accounts in the org, but I don't think any of them actually sync up. Is there a standard / preferred way to test syncing?

Copy link
Contributor

@neob91-close neob91-close left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good, but I recommend doing a smoke test.

@EGrunzke
Copy link
Contributor Author

EGrunzke commented Aug 4, 2025

Changes look good, but I recommend doing a smoke test.

Sync'd up an email and was able to both send and receive.

@EGrunzke EGrunzke merged commit 4cf67a8 into master Aug 4, 2025
4 checks passed
@neob91-close neob91-close deleted the 555-upgrade-requests-to-the-latest-version branch August 5, 2025 07:11
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.

2 participants