Skip to content
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

Turn all remaining TODOs in code base in to GitHub issues: #23

Open
drvinceknight opened this issue Aug 24, 2020 · 0 comments
Open

Turn all remaining TODOs in code base in to GitHub issues: #23

drvinceknight opened this issue Aug 24, 2020 · 0 comments

Comments

@drvinceknight
Copy link
Owner

Currently running:

$ grep --recursive "TODO" .
./tests/test_release.py:    should be fixed. TODO When fixed remove this line of documentation.
./tests/test_release.py:    # TODO Add test that shows wrong behaviour of changing the imported JSON
./tests/test_release.py:    except FileNotFoundError:  # TODO Ensure py3.8 is used so that can pass
./tests/test_release.py:    # TODO Add a better pytest cleanup.
./tests/test_release.py:    except FileNotFoundError:  # TODO Ensure py3.8 is used so that can pass
./tests/nbs/.ipynb_checkpoints/wip-checkpoint.ipynb:    "                assertion = cell[\"source\"][:cell[\"source\"].index(\"\\\"\")]  # TODO Fix this to obtain code check for assertion\n",
Binary file ./tests/__pycache__/test_release.cpython-37-pytest-5.4.3.pyc matches
./tests/test_cli.py:    # TODO Add better tear down.
./tests/test_cli.py:    # TODO Add a better pytest cleanup.
./tests/test_cli.py:    except FileNotFoundError:  # TODO Ensure py3.8 is used so that can pass
./tests/test_cli.py:    # TODO add better tear down.
./tests/test_cli.py:    # TODO Add better tear down.
./tests/test_cli.py:    # assert output.stderr == b''  # TODO Fix the warning error
./tests/test_cli.py:    # TODO Add better tear down.
./docs/_build/html/_modules/nbchkr/utils.html:<span class="p">):</span>  <span class="c1"># TODO Add typing to this function</span>
./docs/_build/html/_modules/nbchkr/utils.html:                        <span class="k">pass</span>  <span class="c1"># TODO Add test coverage for this statement</span>
./docs/_build/html/_modules/nbchkr/utils.html:                <span class="k">pass</span>  <span class="c1"># TODO Add test coverage for this statement</span>
./docs/_build/html/_modules/nbchkr/utils.html:        <span class="c1"># TODO Use the walrus operator here.</span>
./docs/_build/html/_modules/nbchkr/utils.html:    <span class="c1"># TODO I do not really understand why the `[&quot;&quot;, None]` is required here.</span>
./docs/_build/html/_static/jquery-3.5.1.js:                                                     // TODO: identify versions
./docs/_build/html/_static/jquery-3.5.1.js:                                             // TODO: identify versions
./docs/_build/html/_static/jquery-3.5.1.js://   4. _Never_ expose "private" data to user code (TODO: Drop _data, _removeData)
./docs/_build/html/_static/jquery-3.5.1.js:     // TODO: Now that all calls to _data and _removeData have been replaced
Binary file ./docs/_build/doctrees/environment.pickle matches
./src/nbchkr/utils/__init__.py:):  # TODO Add typing to this function
./src/nbchkr/utils/__init__.py:                        pass  # TODO Add test coverage for this statement
./src/nbchkr/utils/__init__.py:                pass  # TODO Add test coverage for this statement
./src/nbchkr/utils/__init__.py:        # TODO Use the walrus operator here.
./src/nbchkr/utils/__init__.py:    # TODO I do not really understand why the `["", None]` is required here.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant