Closed Bug 1504339 Opened 10 months ago Closed 6 months ago
Use tarfile context manager in extract
47 bytes, text/x-phabricator-request
|Details | Review|
https://searchfox.org/mozilla-central/rev/50ba1dd30cf013bddce1ae756f1b3c95b26f0628/testing/mozharness/mozfile/mozfile.py#37 Instead of manually closing, we can use a context manager here.
Hi Marco I would like to take on this bug. However I never use mercurial before. could you guide me on how to submit a patch?
You can follow the guide at https://mozilla-version-control-tools.readthedocs.io/en/latest/hgmozilla/firefoxworkflow.html (but instead of pusing to MozReview using "hg push review", you are going to push to Phabricator using https://github.com/mozilla-conduit/review/blob/master/README.md#execution). Feel free to ping me on IRC if you hit any roadblocks!
Is this something that's being worked on currently, and would it be suitable for someone not so familiar with the codebase? If so, would it be possible for me to work on it?
(In reply to tjdeadma from comment #3) > Is this something that's being worked on currently, and would it be suitable > for someone not so familiar with the codebase? If so, would it be possible > for me to work on it? This is still unassigned, and it's self-contained enough that someone not familiar with the codebase can pick it up. You can definitely work on it if you want. The bug will be assigned once a patch is uploaded.
Assignee: nobody → joseph.bomber
Status: NEW → ASSIGNED
You need to log in before you can comment on or make changes to this bug.