ipython / ipython/comm

Remove dep on traitlet

Open
#15 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
12
Forks
19
PR merge metrics
No merged PRs in 30d

Description

Hi!
as we discovered, the dependency on traitlet >= 5 means that actually, the comm project requires python >= 3.7 contrary to what is published (claims to support python 3.6)

I've seen that the traitlet dependency is used just in order to import this : https://github.com/ipython/traitlets/blob/main/traitlets/utils/importstring.py , which is pretty stable and easy

I suggest to just copy the method in comm package and remove the dep on traitlet all-together
what do you think ?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating where the comm package imports traitlets.utils.importstring and compare it with traitlets' importstring.py implementation linked in the issue. Remove the dependency while preserving the package's stated Python 3.6 support, then run the existing test suite; the issue does not name specific files or tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.