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

Clean up sys.path.append #917

Closed
lintool opened this issue Dec 28, 2021 · 8 comments
Closed

Clean up sys.path.append #917

lintool opened this issue Dec 28, 2021 · 8 comments

Comments

@lintool
Copy link
Member

lintool commented Dec 28, 2021

We have a bunch of these idioms scattered throughout the code base:

import sys
sys.path.append('.')

Since our installation instructions suggest we do:

$ pip install -e .

These can be cleaned up?

@HAKSOAT
Copy link
Contributor

HAKSOAT commented Feb 1, 2022

Hi @lintool can I work on this? I think it's a good first task.

@lintool
Copy link
Member Author

lintool commented Feb 1, 2022

Sure! Any contributions welcome!

@lintool
Copy link
Member Author

lintool commented Mar 6, 2022

hi @HAKSOAT - if you're interested in helping out, there are other small tasks that we could use help on...

@HAKSOAT
Copy link
Contributor

HAKSOAT commented Mar 6, 2022

Hi @lintool Yes, I am very interested. I'd appreciate it if you fill me in.

@lintool
Copy link
Member Author

lintool commented Mar 6, 2022

Please ping me on email?

@HAKSOAT
Copy link
Contributor

HAKSOAT commented Mar 6, 2022

Your uwaterloo email right?

@lintool
Copy link
Member Author

lintool commented Mar 6, 2022

Yes please.

@HAKSOAT
Copy link
Contributor

HAKSOAT commented Mar 6, 2022

Done.

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

2 participants