Replies: 2 comments
-
Target should be part of ".gitignor" and be not committed. Note: we are discussing a big change in the way we run tests and logs - so this will be part of that task. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Regarding ignoring target/, personally I prefer putting a strict
".gitignore" inside the directory (ignore everything except itself, see in
the library's outputs). It guatantees that the directory will exist,
instead of having to recreate it every time.
Regarding readme, it can be somewhere in the repo. I don't see a reason why
generate it as a build output.
…On Thu, Jun 10, 2021, 09:48 Amichai Ben-David ***@***.***> wrote:
Yes - Target should be part of ".gitignor" and be no committed.
We did not enforce it at first, due to problems with running the tests -
and we wanted to review the results.
I will Talk to the Designers and make sure they remove the "trks" from the
target, add it to the ignore, and out a "README" where they describe how to
run the tests and generate the target trks.
Note: we are discussing a big change in the way we run tests and logs - so
this will be part of that task.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#70 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA3P7OE4Q2HFKLC7SEQRE3TTSBN3LANCNFSM46NLXHMQ>
.
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
If I understand correctly, target/ is where execution logs will be written. Is there any reason to commit files there?
Beta Was this translation helpful? Give feedback.
All reactions