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

[tools] Allow to choose the test files when using run-tests.sh #2737

Conversation

gabrieldemarmiesse
Copy link
Contributor

Also rectified a small mistake in the guide: unlike what was written previously, the contributors don't need to run lit after using the script run-tests.sh.

Signed-off-by: gabrieldemarmiesse <gabrieldemarmiesse@gmail.com>
@gabrieldemarmiesse gabrieldemarmiesse requested review from a team as code owners May 18, 2024 11:55
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.

Nice. Thank you!

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

!sync

@JoeLoser JoeLoser added the imported-internally Signals that a given pull request has been imported internally. label May 18, 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 20, 2024
@modularbot
Copy link
Collaborator

Landed in a8f3844! Thank you for your contribution 🎉

modularbot pushed a commit that referenced this pull request May 21, 2024
…ts.sh` (#40225)

[External] [tools] Allow to choose the test files when using
`run-tests.sh`

Also rectified a small mistake in the guide: unlike what was written
previously, the contributors don't need to run `lit` after using the
script `run-tests.sh`.

Co-authored-by: Gabriel de Marmiesse <gabriel.demarmiesse@datadoghq.com>
Closes #2737
MODULAR_ORIG_COMMIT_REV_ID: f0013a6d9c0903bed9392545555810607ec7d8cf
@modularbot modularbot closed this May 21, 2024
martinvuyk pushed a commit to martinvuyk/mojo that referenced this pull request May 24, 2024
…ts.sh` (#40225)

[External] [tools] Allow to choose the test files when using
`run-tests.sh`

Also rectified a small mistake in the guide: unlike what was written
previously, the contributors don't need to run `lit` after using the
script `run-tests.sh`.

Co-authored-by: Gabriel de Marmiesse <gabriel.demarmiesse@datadoghq.com>
Closes modularml#2737
MODULAR_ORIG_COMMIT_REV_ID: f0013a6d9c0903bed9392545555810607ec7d8cf
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