https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=242616
Kubilay Kocak <ko...@freebsd.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|[NEW PORT] |[NEW PORT]
|www/py-flask-babelex: |www/py-flask-babelex: Adds
|i18n/l10n support to Flask |i18n/l10n support to Flask
|applications with the help |applications
|of the Babel library |
--- Comment #3 from Kubilay Kocak <ko...@freebsd.org> ---
Review items (already fixed locally):
- COMMENT should match setup.py:description (modulo portlint compliance)
- setup.py:*_requires declares version requirements, match these as closely as
possible, given our *_DEPENDS syntax:
install_requires=[
'Flask',
'Babel>=1.0',
'speaklater>=1.2',
'Jinja2>=2.5'
],
- Port Makefile calls pytest but doesn't TEST_DEPEND on pytest
- Doesn't appear project requires pytest (calls `python tests.py` [1])
- Add comments to patches wherever possible, ideally including upstream
issue/PR/commit urls where applicable
[1] Makefile: @cd tests; python tests.py
--
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
freebsd-python@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-python
To unsubscribe, send any mail to "freebsd-python-unsubscr...@freebsd.org"
