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

Update stability info in documentation #3216

Merged
merged 5 commits into from
Feb 28, 2025
Merged

Update stability info in documentation #3216

merged 5 commits into from
Feb 28, 2025

Conversation

mikenerone
Copy link
Member

This PR updates the README to better reflect Trio’s current maturity. The previous wording hadn’t been updated since it was originally written eight years ago and no longer accurately represented Trio’s suitability for production use.

No functional changes—just a documentation update for clarity.

@mikenerone mikenerone requested a review from njsmith February 26, 2025 17:35
Copy link
Member

@CoolCat467 CoolCat467 left a comment

Choose a reason for hiding this comment

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

Looks solid to me

@A5rocks
Copy link
Contributor

A5rocks commented Feb 26, 2025

Could you update the docs? They have this paragraph too.

@mikenerone
Copy link
Member Author

mikenerone commented Feb 26, 2025

@A5rocks

Could you update the docs? They have this paragraph too.

Good call, updated several places in the docs.

@agronholm
Copy link
Contributor

How about also updating this line?

@mikenerone mikenerone changed the title Update stability info in README Update stability info in documentation Feb 26, 2025
@mikenerone
Copy link
Member Author

mikenerone commented Feb 26, 2025

@agronholm

How about also updating this line?

Done!

Copy link

codecov bot commented Feb 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00000%. Comparing base (4df83b4) to head (5ac6aeb).
Report is 6 commits behind head on main.

Additional details and impacted files
@@               Coverage Diff               @@
##                 main        #3216   +/-   ##
===============================================
  Coverage   100.00000%   100.00000%           
===============================================
  Files             124          124           
  Lines           18770        18770           
  Branches         1269         1269           
===============================================
  Hits            18770        18770           
Files with missing lines Coverage Δ
src/trio/_deprecate.py 100.00000% <ø> (ø)

Copy link
Contributor

@A5rocks A5rocks left a comment

Choose a reason for hiding this comment

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

Thanks for working through the wording. I'd like @njsmith to weigh in before we merge this.

@jakkdl jakkdl requested a review from Zac-HD February 27, 2025 10:28
Copy link
Member

@Zac-HD Zac-HD left a comment

Choose a reason for hiding this comment

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

lgtm - Anthropic has been using Trio in production for years now, and we're very happy with it 😁

@jakkdl jakkdl added this pull request to the merge queue Feb 28, 2025
Merged via the queue into main with commit a78b5da Feb 28, 2025
81 checks passed
@jakkdl jakkdl deleted the update-readme-stability branch February 28, 2025 22:13
@njsmith
Copy link
Member

njsmith commented Feb 28, 2025

lgtm

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.

8 participants