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

Upgrade prompt_toolkit to 3.0.43 #37548

Merged
1 commit merged into from
Mar 10, 2024
Merged

Conversation

kwankyu
Copy link
Collaborator

@kwankyu kwankyu commented Mar 6, 2024

Fixes #37539.

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@kwankyu kwankyu changed the title Upgrade prompt-toolkit to 3.0.43 Upgrade prompt_toolkit to 3.0.43 Mar 6, 2024
@kwankyu
Copy link
Collaborator Author

kwankyu commented Mar 6, 2024

According to the upcoming standard, set back to "critical".

@mkoeppe
Copy link
Member

mkoeppe commented Mar 6, 2024

You mean until "positive review" is set?

@mkoeppe
Copy link
Member

mkoeppe commented Mar 6, 2024

Unproblematic upgrade, and already reported it #37539 to fix the problem.
So let's get this in.

Copy link

github-actions bot commented Mar 6, 2024

Documentation preview for this PR (built with commit c38b7ab; changes) is ready! 🎉

@kwankyu
Copy link
Collaborator Author

kwankyu commented Mar 6, 2024

Yes. Thanks.

@mkoeppe mkoeppe added this to the sage-10.3 milestone Mar 6, 2024
vbraun pushed a commit to vbraun/sage that referenced this pull request Mar 8, 2024
    
<!-- ^ Please provide a concise and informative title. -->
<!-- ^ Don't put issue numbers in the title, do this in the PR
description below. -->
<!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method
to calculate 1 + 2". -->
<!-- v Describe your changes below in detail. -->
<!-- v Why is this change required? What problem does it solve? -->
<!-- v If this PR resolves an open issue, please link to it here. For
example, "Fixes sagemath#12345". -->

Fixes sagemath#37539.

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation accordingly.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
    
URL: sagemath#37548
Reported by: Kwankyu Lee
Reviewer(s):
@vbraun vbraun closed this pull request by merging all changes into sagemath:develop in e417e22 Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"DeprecationWarning: There is no current event loop" at startup
3 participants