Skip to content

Implement AsMapping for PyStr#3417

Merged
youknowone merged 1 commit into
RustPython:mainfrom
moreal:as-mapping-str
Nov 6, 2021
Merged

Implement AsMapping for PyStr#3417
youknowone merged 1 commit into
RustPython:mainfrom
moreal:as-mapping-str

Conversation

@moreal

@moreal moreal commented Nov 6, 2021

Copy link
Copy Markdown
Contributor

This pull request tries to resolve the part of #3244.

Mapping trait to provide type slot as_mapping

P.S. maybe it should be AsMapping trait 🤔

@moreal moreal marked this pull request as ready for review November 6, 2021 07:27
@youknowone

Copy link
Copy Markdown
Member

@Snowapril could you review this PR?

@Snowapril Snowapril left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me 😊

@youknowone youknowone merged commit 820a67c into RustPython:main Nov 6, 2021
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.

3 participants