gitignore: Ignore myresults.xml files #72870
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
omkar3141
requested review from
fabiobaltieri,
ludvigsj,
carlescufi,
alxelax and
Andrewpini
and removed request for
fabiobaltieri
fabiobaltieri
previously approved these changes
May 16, 2024
nashif
previously approved these changes
May 16, 2024
nashif
added
the
Trivial
aescolar
reviewed
May 16, 2024
There was a problem hiding this comment.
Kind of amusing how a "whatever" name in a documentation example ends up being used by many.
Anyhow, the default for this is actually "RunResults.xml" one level above the zephyr repo.
So maybe we should just change the doc to say "RunResults.xml" instead and have this be the one we ignore?
Anyhow, this can also be merged as is, it does no harm.
.gitignore
Outdated
| @@ -85,3 +85,6 @@ ModulesMaintainers.txt | |||
| Nits.txt | |||
| Pylint.txt | |||
| YAMLLint.txt | |||
|
|
|||
| # Local test run outputs | |||
| myresults.xml | |||
There was a problem hiding this comment.
:), I would put it next to the other tests results in line 40ish
These files are created as a result of local babblesim test runs and can be safely ignored. Signed-off-by: Omkar Kulkarni <omkar.kulkarni@nordicsemi.no>
omkar3141
force-pushed
the
omku/ignore_results_xml_files
branch
from
cc15410
to
772f370
Compare
aescolar
approved these changes
May 27, 2024
fabiobaltieri
approved these changes
May 27, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These files are created as a result of local babblesim test runs and can be safely ignored.