Bug 256187 - [NEW-PORT] www/py-django-ckeditor: Django admin CKEditor integration
Summary: [NEW-PORT] www/py-django-ckeditor: Django admin CKEditor integration
Status: Closed Feedback Timeout
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Fernando Apesteguía
URL: https://pypi.org/project/django-ckedi...
Keywords: feature, needs-patch, needs-qa
Depends on:
Blocks:
 
Reported: 2021-05-27 11:36 UTC by gettoknowmi
Modified: 2023-08-09 14:21 UTC (History)
5 users (show)

See Also:
koobs: maintainer-feedback? (gettoknowmi)


Attachments
py-django-ckeditor port files (1.27 KB, application/x-shellscript)
2021-05-27 11:36 UTC, gettoknowmi
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description gettoknowmi 2021-05-27 11:36:01 UTC
Created attachment 225306 [details]
py-django-ckeditor port files

https://pypi.org/project/django-ckeditor/
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2021-06-02 09:25:25 UTC
^Triage: fix Summary.

Note to submitter: the pkg-descr is kind of sparse.  To me, it seems to assume that I already know what the package does.
Comment 2 gettoknowmi 2021-07-30 14:58:07 UTC
do i need to provide anything else to get this in the port tree?
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2021-07-31 00:16:00 UTC
Review items:

 - Latest upstream version is 6.1.0
 - Add python to CATEGORIES
 - Match COMMENT to upstream description (setup.py:description). "Django admin CKEditor integration"
 - Match upstream dependency <versions-specs> ("django-js-asset>=1.2.2",)
 - ",run" is incorrect for most python package cases. It excludes "build" ('run time *only*), which is required to compile py files to pyo pyc counterparts. 
 - Upstream is only testing Python 3.6+, this port declares 3.4+ support. This isd fine if the packages tests pass with all allowed versions.
 - Upstream ships tests. Add TEST_DEPENDS and a test-target for comprehensive QA:

Note #1: we don't need to run tox, coverage or lint tests, the following two should be sufficient:

TEST_ENV+=DJANGO_SETTINGS_MODULE=ckeditor_demo.settings

  ${SETENV} ${TEST_ENV} ${PYTHON_CMD} manage.py test ckeditor_demo 
  ${SETENV} ${TEST_ENV} ${PYTHON_CMD} manage.py validate_templates

Note 2: There are a few more dependencies for tests (pillow, django_extensions). Only add compulsory ones. Interesting to note that these deps arent in install_requires. Are they 'optional' ? Maybe its worth having OPTIONS for these.

Confirmation that the port passes tests (portlint, poudriere) would also be great.
Comment 4 Kubilay Kocak freebsd_committer freebsd_triage 2021-07-31 00:16:28 UTC
See Also: https://wiki.freebsd.org/Python/PortsPolicy