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

[stdlib] add Defaultable trait #2526

Closed
wants to merge 1 commit into from

Conversation

helehex
Copy link
Contributor

@helehex helehex commented May 5, 2024

adds a Defaultable trait for describing a type which has a constructor with no arguments

@helehex helehex requested a review from a team as a code owner May 5, 2024 11:58
```
"""

fn __init__(inout self):
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we have the name default

@ematejska ematejska added the mojo-repo Tag all issues with this label label May 6, 2024
Copy link
Collaborator

@JoeLoser JoeLoser left a comment

Choose a reason for hiding this comment

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

LGTM! We can bikeshed the name another time. I'll bring it up in Monday's team meeting.

@JoeLoser JoeLoser added the imported-internally Signals that a given pull request has been imported internally. label May 10, 2024
Signed-off-by: Max Brylski <[email protected]>
@helehex
Copy link
Contributor Author

helehex commented May 13, 2024

i simplified the docstring

@JoeLoser JoeLoser self-assigned this May 14, 2024
@modularbot
Copy link
Collaborator

✅🟣 This contribution has been merged 🟣✅

Your pull request has been merged to the internal upstream Mojo sources. It will be reflected here in the Mojo repository on the nightly branch during the next Mojo nightly release, typically within the next 24-48 hours.

We use Copybara to merge external contributions, click here to learn more.

@modularbot modularbot added the merged-internally Indicates that this pull request has been merged internally label May 14, 2024
modularbot pushed a commit that referenced this pull request May 14, 2024
[External] [stdlib] Add `Defaultable` trait

Add a `Defaultable` trait for describing a type which has a constructor
with no arguments.

Co-authored-by: Helehex <[email protected]>
Closes #2526
MODULAR_ORIG_COMMIT_REV_ID: aed69a25ecf531873f944466a1cb5d7652dd7425
@modularbot modularbot added the merged-externally Merged externally in public mojo repo label May 14, 2024
@modularbot
Copy link
Collaborator

Landed in 60f5b04! Thank you for your contribution 🎉

@modularbot modularbot closed this May 14, 2024
rd4com pushed a commit to rd4com/mojo_branch that referenced this pull request May 15, 2024
[External] [stdlib] Add `Defaultable` trait

Add a `Defaultable` trait for describing a type which has a constructor
with no arguments.

Co-authored-by: Helehex <[email protected]>
Closes modularml#2526
MODULAR_ORIG_COMMIT_REV_ID: aed69a25ecf531873f944466a1cb5d7652dd7425

Signed-off-by: rd4com <[email protected]>
lsh pushed a commit to lsh/mojo that referenced this pull request May 17, 2024
[External] [stdlib] Add `Defaultable` trait

Add a `Defaultable` trait for describing a type which has a constructor
with no arguments.

Co-authored-by: Helehex <[email protected]>
Closes modularml#2526
MODULAR_ORIG_COMMIT_REV_ID: aed69a25ecf531873f944466a1cb5d7652dd7425

Signed-off-by: Lukas Hermann <[email protected]>
martinvuyk pushed a commit to martinvuyk/mojo that referenced this pull request May 24, 2024
[External] [stdlib] Add `Defaultable` trait

Add a `Defaultable` trait for describing a type which has a constructor
with no arguments.

Co-authored-by: Helehex <[email protected]>
Closes modularml#2526
MODULAR_ORIG_COMMIT_REV_ID: aed69a25ecf531873f944466a1cb5d7652dd7425
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
imported-internally Signals that a given pull request has been imported internally. merged-externally Merged externally in public mojo repo merged-internally Indicates that this pull request has been merged internally mojo-repo Tag all issues with this label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants