Skip to content

Conversation

@BharatAtbrat
Copy link
Contributor

@BharatAtbrat BharatAtbrat commented Aug 13, 2024

sonnyp
sonnyp previously requested changes Aug 13, 2024
Copy link
Contributor

@sonnyp sonnyp left a comment

Choose a reason for hiding this comment

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

Please move to a file in modules/gom.json and use the http archive
like other dependencies such as https://github.com/workbenchdev/Workbench/blob/main/build-aux/modules/libshumate.json

create gom.json for dependencies
remove redundant use in devel
@BharatAtbrat BharatAtbrat requested a review from sonnyp August 13, 2024 18:52
@andyholmes andyholmes dismissed sonnyp’s stale review September 5, 2024 05:41

Requested changes have been made, so proceeding with merge.

@andyholmes andyholmes merged commit 154f5ea into main Sep 5, 2024
@andyholmes andyholmes deleted the gom_dep branch September 5, 2024 05:42
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.

4 participants