Comcast / Comcast/sheens

Remove core dependency on encoding.json

Open
#14 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
122
Forks
27
PR merge metrics
No merged PRs in 30d

Description

Rationale: Reduce footprint.

Also see #13.

Right now, the `encoding.json` dependencies are minimal. The most important use is for `Spec.PatternSyntax` and related code, which knows how to parse branch patterns rendered in JSON.

Approach: Support pluggable pattern parsers.

Contributor guide

Open the contributing guide

Research direction

Start by tracing the core uses of encoding.json, especially Spec.PatternSyntax and the related branch-pattern parsing mentioned in the issue. Read issue #13 for context and map the current dependency boundary. Done means pattern parsers are pluggable and the core no longer depends on encoding.json.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.