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

Unify the adding of libs into the path in one place #9251

Closed
vojtechjelinek opened this issue May 5, 2020 · 4 comments
Closed

Unify the adding of libs into the path in one place #9251

vojtechjelinek opened this issue May 5, 2020 · 4 comments

Comments

@vojtechjelinek
Copy link
Member

Currently, we add the third party libs directories to sys.path in multiple files:

  • appengine_config.py
  • core/tests/gae_suite.py
  • scripts/linters/pre_commit_linter.py
  • scripts/run_backend_tests.py

unify this into one place so the updates to libraries are more easy to do.

@tanishkab
Copy link

Please assign this issue to me. Thank You in advance

@vojtechjelinek
Copy link
Member Author

@Tanishka-Bansal Thanks for the interest, please go through these instructions first https://github.com/oppia/oppia/wiki/Contributing-code-to-Oppia and then you can try some issue with good first issue label.

@oppiabot
Copy link

oppiabot bot commented May 24, 2021

Hi @oppia/core-maintainers, this issue is not assigned to any project. Can you please update the same? Thanks!

@U8NWXD U8NWXD added this to Triage in Developer experience Jan 6, 2022
@U8NWXD
Copy link
Member

U8NWXD commented Aug 1, 2022

We no longer need to change the PATH so much now that we have #15602, so I don't think this is relevant anymore

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants