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

setup instructions in CONTRIBUTING.md do not install all dependencies #1150

Open
EmilyBStudent opened this issue Nov 17, 2024 · 3 comments · May be fixed by #1159
Open

setup instructions in CONTRIBUTING.md do not install all dependencies #1150

EmilyBStudent opened this issue Nov 17, 2024 · 3 comments · May be fixed by #1159

Comments

@EmilyBStudent
Copy link

Overview Description

I followed the instructions in CONTRIBUTING.md to clone the Babel repository and set up a development environment, but when I ran "make test" as the final step, several thousand of the tests failed. It appeared that zoneinfo/pytz and tzdata were not installed in the virtual environment. After installing them manually I was able to run the tests successfully.

Reproducibility

Always.

Additional Information

This is using Python 3.11 on Windows 11. Modules installed by “pip install -e .[dev]” were:

  • six
  • pluggy
  • packaging
  • iniconfig
  • coverage
  • colorama
  • babel
  • python-dateutil
  • pytest
  • pytest-cov
  • freezegun
@tomasr8
Copy link
Member

tomasr8 commented Nov 17, 2024

Interesting, quoting from https://docs.python.org/3/library/zoneinfo.html#data-sources

[...] Some systems, including notably Windows systems, do not have an IANA database available, and so for projects targeting cross-platform compatibility that require time zone data, it is recommended to declare a dependency on tzdata.

We actually already require tzdata in tests but not as a project dependency:

babel/tox.ini

Line 16 in 740ee3d

tzdata;sys_platform == 'win32'

I suppose we could add tzdata as a dependency for Windows? @akx thoughts?

@akx
Copy link
Member

akx commented Nov 17, 2024

Sure! It'd be better to have the test (dev) deps in one place only, IOW replace the deps in tox.ini with extras = dev and make sure the extras_require list in setup.py is in sync with that.

@wandrew004
Copy link

Can I work on this?

@wandrew004 wandrew004 linked a pull request Dec 5, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants