Recent updates to `Dossier.prepare_dataframes`
Maintainers usually reply within 1 day
A pull request for this has already been merged.
- #173 by @amotl — merged
Assessment
- Difficulty
- 2/5
- Estimated time
- Under an hour
- Newbie friendliness
- 70/100
Research direction
Open patzilla/navigator/export.py and inspect lines 97-109 in Dossier.prepare_dataframes to find the unused lst variable from commit 9f2ae3b. Determine whether to remove it or integrate it into the dataframe setup. Done when lst is either used or deleted and existing tests pass.
Written by the indexing model from the issue text.
Description
Dear @papoteur-mga,
thank you for helping to improve the codebase. Regarding your recent commit 9f2ae3b47cb3caa07c0b2f704e7fb1295aeaa93c, I can see the newly introduced lst variable is never used yet. Is it possible that the fix wasn't complete?
With kind regards,
Andreas.
- Dominant language
- Python
- Stars
- 123
- Forks
- 31
- Avg merge
- 3h 7m
- Merged PRs (30d)
- 71
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from ip-tools/patzilla
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
ip-tools/patzilla#139 · 3 comments ·
Maintainers usually reply within 1 day
-
UI: Update BabelOpendependencies help wanted javascript
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
Maintainers usually reply within 1 day
-
dependencies help wanted javascript
Difficulty 2/5 1-3 hours Newbie friendliness 45/100
ip-tools/patzilla#181 · 1 comment ·
Maintainers usually reply within 1 day
-
dependencies javascript
Difficulty 5/5 Over a week Newbie friendliness 15/100
Maintainers usually reply within 1 day
-
dependencies javascript
Difficulty 5/5 Over a week Newbie friendliness 18/100
ip-tools/patzilla#176 · 1 comment ·
Maintainers usually reply within 1 day
All issues in ip-tools/patzilla
Similar issues
-
cost:cheap practicality:seen-in-the-wild severity:medium
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
fairagro/m4.2_advanced_middleware_api#620 ·
Maintainers usually reply within 1 day
-
area:security effort:S priority:P2 provider:local-libvirt status:ready type:docs
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
randomparity/kdive#3495 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
microsoft/markitdown#2820 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 2 days