-
Notifications
You must be signed in to change notification settings - Fork 77
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
Manual thunderbird tinderbox test_case is wrong #238
Comments
Agreed. Marking as a mentored issue then. |
I maybe be that newcomer Nebelhom is referring to. If this issue still needs attention. |
Oh, this was actually done via PR #402. Looks like we simply forgot to mark this issue as closed. @andreheringer if you are still interested to help let me know and we can certainly find another spot to get you started. |
@whimboo oh ok. Sure I would love to help, have 2 years of experience working in python but never contribute to open source. Yet I hope to be helpful. |
@andreheringer this is great. Would you mind having a look at issue #550 which just needs some tests to be disabled? It's an easy fix, and will help you to get into the project, and how to run the tests. Thanks! |
I just noticed earlier that the following manual test case in
test_cases.md
is wrongmozdownload -a thunderbird -t tinderbox -p linux --branch=mozilla-central --extension=txt
It should be
--branch=comm-central
Unless we are implementing this test_case any time soon, I would say this is a great first patch for a newcomer.
The text was updated successfully, but these errors were encountered: