Skip to content

t-strings: Various changes#74

Merged
lysnikolaou merged 6 commits intolysnikolaou:tstringsfrom
AA-Turner:tstrings-misc
Apr 23, 2025
Merged

t-strings: Various changes#74
lysnikolaou merged 6 commits intolysnikolaou:tstringsfrom
AA-Turner:tstrings-misc

Conversation

@AA-Turner
Copy link
Copy Markdown

@AA-Turner AA-Turner commented Apr 22, 2025

cc @lysnikolaou

Various changes, probably best reviewed commit-by-commit:

  • Update include directive comments to reflect the compiler changes
  • Remove unneeded stackrefs include in a header file
  • Rename ftstring_parts to parts in the AST unparser
  • Use sets for membership testing
  • Update documentation
  • Remove outdated entries from cases_generator/analyzer.py

📚 Documentation preview 📚: https://cpython-previews--74.org.readthedocs.build/

@AA-Turner AA-Turner requested a review from lysnikolaou as a code owner April 22, 2025 22:44
Copy link
Copy Markdown
Owner

@lysnikolaou lysnikolaou left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks, @AA-Turner!

@lysnikolaou lysnikolaou merged commit 8842739 into lysnikolaou:tstrings Apr 23, 2025
34 checks passed
@AA-Turner AA-Turner deleted the tstrings-misc branch April 23, 2025 09:51
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