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

[Docs] correct output, fix indent in Heterogeneous variadic arguments #2741

Closed
wants to merge 2 commits into from

Conversation

kengz
Copy link

@kengz kengz commented May 18, 2024

The current Heterogeneous variadic arguments (see screenshot) - last example has wrong output (Bob0 should be Bob), and indentation of the example code is also off (had to edit after copying into Mojo playground)

Screenshot 2024-05-12 at 9 31 51 AM

Signed-off-by: Wah Loon Keng <kengzwl@gmail.com>
@kengz kengz requested a review from a team as a code owner May 18, 2024 17:05
@KenJones-Modular
Copy link

!sync

@modularbot modularbot added the imported-internally Signals that a given pull request has been imported internally. label May 22, 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 merged-internally Indicates that this pull request has been merged internally merged-externally Merged externally in public mojo repo labels May 22, 2024
@modularbot
Copy link
Collaborator

Landed in a2c9820! Thank you for your contribution 🎉

modularbot pushed a commit that referenced this pull request May 23, 2024
…c arguments (#40435)

[External] [Docs] correct output, fix indent in Heterogeneous variadic
arguments

The current [Heterogeneous variadic
arguments](https://docs.modular.com/mojo/manual/functions#heterogeneous-variadic-arguments)
(see screenshot) - last example has wrong output (`Bob0` should be
`Bob`), and indentation of the example code is also off (had to edit
after copying into Mojo playground)

<img width="838" alt="Screenshot 2024-05-12 at 9 31 51 AM"
src="https://github.com/modularml/mojo/assets/8209263/9ea82a34-9ac4-4414-b391-9d8b7f31357f">

Co-authored-by: Wah Loon Keng <kengzwl@gmail.com>
Closes #2741
MODULAR_ORIG_COMMIT_REV_ID: c06231809a24351548c4967f66faf65d9204e859
@modularbot modularbot closed this May 23, 2024
martinvuyk pushed a commit to martinvuyk/mojo that referenced this pull request May 24, 2024
…c arguments (#40435)

[External] [Docs] correct output, fix indent in Heterogeneous variadic
arguments

The current [Heterogeneous variadic
arguments](https://docs.modular.com/mojo/manual/functions#heterogeneous-variadic-arguments)
(see screenshot) - last example has wrong output (`Bob0` should be
`Bob`), and indentation of the example code is also off (had to edit
after copying into Mojo playground)

<img width="838" alt="Screenshot 2024-05-12 at 9 31 51 AM"
src="https://github.com/modularml/mojo/assets/8209263/9ea82a34-9ac4-4414-b391-9d8b7f31357f">

Co-authored-by: Wah Loon Keng <kengzwl@gmail.com>
Closes modularml#2741
MODULAR_ORIG_COMMIT_REV_ID: c06231809a24351548c4967f66faf65d9204e859
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants