pimutils / pimutils/vdirsyncer
pair configuration: `collections = null` is not a good default.
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 1.9k
- Forks
- 185
- Avg merge
- 11h 49m
- Merged PRs (30d)
- 1
Description
null implies that the user entered direct URLs and paths to a collection. This is useful behavior in the case where collection discovery is completely broken on the server, but it is only rarely used and therefore a bad default. We have two options: PLAN:
- Require a value, no default done in 0.9.0
- Change the default, I suggest
["from a", "from b"]1.0
Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
Contributor guide
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 by tracing how the collections configuration value is parsed and where its default is defined. Check the existing configuration tests, if present, then verify that omitting the value uses the intended non-null default while direct URLs and paths still work; update the behavior for the 1.0 plan.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- cli
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100