Skip to content

Conversation

@kianmeng
Copy link
Contributor

DESCRIPTION_HERE

Found via codespell -S *.svg -L fo,nd,filp,te,iif,4rd and typos --hidden --format brief

Closes #ISSUE


Contributor checklist:

  • I have read and understood CONTRIBUTING.md
  • Confirmed an issue exists for the PR, and the text Closes #issue appears in the PR summary (e.g., Closes #123).
  • Confirmed PR is rebased onto the latest base
  • Confirmed failure before change and success after change
  • Any generic new functionality is replicated across cloud providers if necessary
  • Tested manually against live server backend for at least one provider
  • Added tests for any new functionality
  • Linting passes locally
  • Tests pass locally
  • Updated HISTORY.md with the issue that is addressed and the PR you are submitting. If the top section is not `## UNRELEASED``, then you need to add a new section to the top of the document for your change.

Found via `codespell -S *.svg -L fo,nd,filp,te,iif,4rd`
and `typos --hidden --format brief`
@jayqi jayqi self-requested a review March 13, 2025 19:11
@codecov
Copy link

codecov bot commented Mar 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.3%. Comparing base (d3522c7) to head (a377a2a).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #512     +/-   ##
========================================
- Coverage    93.7%   92.3%   -1.4%     
========================================
  Files          23      23             
  Lines        1798    1798             
========================================
- Hits         1685    1661     -24     
- Misses        113     137     +24     
Files with missing lines Coverage Δ
cloudpathlib/client.py 88.1% <ø> (ø)
cloudpathlib/s3/s3client.py 92.9% <ø> (-1.5%) ⬇️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@jayqi jayqi left a comment

Choose a reason for hiding this comment

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

Thanks @kianmeng!

@jayqi jayqi merged commit fb81872 into drivendataorg:master Mar 13, 2025
22 of 24 checks passed
@kianmeng
Copy link
Contributor Author

🥳 🥳 🥳 🥳 🥳

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