Skip to content

Fix discovery of Go direct dependencies#6

Merged
wescarr merged 3 commits intomainfrom
fix-go-direct-deps
Sep 6, 2022
Merged

Fix discovery of Go direct dependencies#6
wescarr merged 3 commits intomainfrom
fix-go-direct-deps

Conversation

@wescarr
Copy link
Member

@wescarr wescarr commented Sep 4, 2022

  • Instead of relying on main.go existing in root directory we use go list to list the direct dependencies and then continue with the existing approach to get indirect dependencies

- Instead of relying on main.go existing in root directory we use go list to list the direct dependencies and then continue with the existing approach to get indirect dependencies
@wescarr wescarr requested a review from dudleycarr September 4, 2022 19:50
@wescarr wescarr changed the title Fix discovery of direct dependencies Fix discovery of Go direct dependencies Sep 4, 2022
- Our current approach relied on a main.go file residing at the root of the repository which may not always be the case.

- Additionally calling go list {{.Deps}} module would not always give indented results.
@wescarr wescarr merged commit 4e72843 into main Sep 6, 2022
@wescarr wescarr deleted the fix-go-direct-deps branch September 6, 2022 20:11
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.

1 participant