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

Add String.add iso proposal #185

Merged
merged 1 commit into from
May 4, 2021
Merged

Conversation

ergl
Copy link
Member

@ergl ergl commented Apr 19, 2021

Rendered

For more context see also the Zulip discussion where this was raised.

@SeanTAllen
Copy link
Member

I'm in favor of this change.

@jemc jemc added the status - final comment period The RFC is finalized. Waiting for final comments. label Apr 27, 2021
@jemc jemc added status - ready for merge This RFC was approved and is ready to be merged. Waiting for bot... and removed status - final comment period The RFC is finalized. Waiting for final comments. labels May 4, 2021
@jemc
Copy link
Member

jemc commented May 4, 2021

This RFC was approved in today's sync call.

@SeanTAllen
Copy link
Member

@Theodus when you have time, can you do the dance for this RFC?

@Theodus
Copy link
Contributor

Theodus commented May 4, 2021

@Theodus when you have time, can you do the dance for this RFC?

Sure, I'll take care of it later today

@Theodus Theodus merged commit c089fce into ponylang:main May 4, 2021
@ergl ergl deleted the ergl/string_add_iso branch May 5, 2021 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status - ready for merge This RFC was approved and is ready to be merged. Waiting for bot...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants