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

Affected Issue(s): Fixes when try to run within Windows OS environment #268

Merged
merged 3 commits into from
Aug 23, 2024
Merged

Affected Issue(s): Fixes when try to run within Windows OS environment #268

merged 3 commits into from
Aug 23, 2024

Conversation

muhammad-levi
Copy link
Contributor

Resolves https://github.com/sid-indonesia/it-team/issues/313

What this commit has achieved:

  1. Renamed an entry in .env from username to user_name to accomodate usage within Windows OS (there is a default OS env var %username% within Windows OS)
  2. Added python-magic-bin to the requirements.txt, to resolve issue of:
ImportError: failed to find libmagic.  Check your installation

(Possibly only happens in Windows OS, not really sure)

Resolves sid-indonesia/it-team#313

What this commit has achieved:
1. Renamed an entry in `.env` from `username` to `user_name` to accomodate usage within Windows OS (there is a default OS env var %username% within Windows OS)
2. Added `python-magic-bin` to the `requirements.txt`, to resolve issue of:
```
ImportError: failed to find libmagic.  Check your installation
```
(Possibly only happens in Windows OS, not really sure)
@MargaretNjenga MargaretNjenga requested a review from Wambere August 23, 2024 07:08
importer/requirements.txt Outdated Show resolved Hide resolved
Resolves sid-indonesia/it-team#313

What this commit has achieved:
1. Only add `python-magic-bin` as one of the requirements if the `sys_platform` is Windows.
@Wambere
Copy link
Contributor

Wambere commented Aug 23, 2024

@muhammad-levi please also change username to user_name here in the pytest.ini file for the tests to work

Resolves sid-indonesia/it-team#313

What this commit has achieved:
1. Renamed env var `username` to `user_name` in `pytest.ini`.
2. Corrected the order of default values in the `README.md` of `importer`.
@Wambere Wambere merged commit 11b7fc3 into onaio:main Aug 23, 2024
4 checks passed
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

Successfully merging this pull request may close these issues.

2 participants