Skip to content

Add list-deps sub-command - #136

Merged
Sam Dietrich (xmaams) merged 8 commits into
mainfrom
dev/xmaams/list-deps
Aug 27, 2026
Merged

Add list-deps sub-command#136
Sam Dietrich (xmaams) merged 8 commits into
mainfrom
dev/xmaams/list-deps

Conversation

@xmaams

Copy link
Copy Markdown
Contributor

Adds a list-deps subcommand for dependency resolution (such as in cmake) without rebuild.
Also add a small test case vs a golden file

@xmaams

Copy link
Copy Markdown
Contributor Author

Sam Dietrich (Sam Dietrich (@xmaams)) please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Microsoft"

@blakepelton blakepelton (blakepelton) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, although I'm not the Haskel expert. Did you write this code yourself? One additional test you could add is as script that compares the --list-deps output against the --file-list output.

@xmaams

Copy link
Copy Markdown
Contributor Author

LGTM, although I'm not the Haskel expert. Did you write this code yourself? One additional test you could add is as script that compares the --list-deps output against the --file-list output.

Added a test to compare vs compile --file-list.
Copilot helped with the Haskell, which is probably for the best... But I did read it closely and it looks fine to me with my limited knowledge. I checked that dependency lists did not change between this build and main.

@xmaams
Sam Dietrich (xmaams) merged commit ac73081 into main Aug 27, 2026
24 checks passed
@xmaams
Sam Dietrich (xmaams) deleted the dev/xmaams/list-deps branch August 27, 2026 18:25
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