Skip to content

Add dependency resolution for python#20

Merged
epilande merged 9 commits intoepilande:mainfrom
yuchenliu15:add_py_dep
Jun 8, 2025
Merged

Add dependency resolution for python#20
epilande merged 9 commits intoepilande:mainfrom
yuchenliu15:add_py_dep

Conversation

@yuchenliu15
Copy link
Copy Markdown
Contributor

Adding a PyResolver and its test cases

image

cmd/grab/main.go Outdated
flag.StringVar(&formatName, "f", "markdown", formatUsage+" (shorthand)")

flag.BoolVar(&resolveDeps, "deps", false, "Automatically include direct dependencies (Go, TS/JS)")
flag.BoolVar(&resolveDeps, "deps", false, "Automatically include direct dependencies (Go, TS/JS, PY)")
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
flag.BoolVar(&resolveDeps, "deps", false, "Automatically include direct dependencies (Go, TS/JS, PY)")
flag.BoolVar(&resolveDeps, "deps", false, "Automatically include direct dependencies (Go, TS/JS, Python)")

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed and added in help text as well

grab Outdated
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's remove the binary ✂️

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops, good catch!

Copy link
Copy Markdown
Owner

@epilande epilande left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM! Thanks!!

@epilande epilande merged commit 1006bb5 into epilande:main Jun 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants