OpenPrinting / OpenPrinting/system-config-printer
Just use cups.require() in one place
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 204
- Forks
- 108
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 9
Description
Currently, there are calls to cups.require() in lots of places, which makes it hard to know which version of pycups is really required. It would be much nicer to just have it in, say, system-config-printer.py.
We could also remove lots of compatibility code for older pycups versions. git grep '#.*pycups' shows where this is.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in system-config-printer.py and use git grep 'cups.require()' and git grep '#.*pycups' to map the current calls and compatibility code. Consolidate the requirement in the proposed entry point and remove the obsolete compatibility code where appropriate; done means the required pycups version is clear from one place.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100