bobg / bobg/fab

Try to get away with using only go/*, not golang.org/x/tools/go/packages

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

Description

The code for loading the user's package uses golang.org/x/tools/go/packages. We redundantly load doc strings with go/parser. Maybe we can get away with just one or the other, not both.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at the code that loads the user's package with golang.org/x/tools/go/packages, then inspect where go/parser is used to load documentation strings. Determine whether either loader can provide everything required and verify the result with the existing project checks. Done means the redundant package-loading approach is removed without losing needed package or documentation information.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.