Skip to content

Use taskfile variable to define primary Go module path #141

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 16, 2021
Merged

Use taskfile variable to define primary Go module path #141

merged 1 commit into from
Aug 16, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Aug 15, 2021

Although support was added to the Go module related "templates" for projects which contain modules at other paths (#136), the assumption remained that there will always be a primary module in the root of the repository, and this path was hardcoded at several locations in the taskfiles.

It might be that a project that is not a Go module contains an accessory Go module in a subfolder (example). In this case, the use
of a taskfile variable will allow that path to be defined in one place, avoiding the need to override the hardcoded incorrect location whenever a Go module-related task is run manually.

Although support was added to the Go module related "templates" for projects which contain modules at other paths, the
assumption remained that there will always be a primary module in the root of the repository, and this path was hardcoded
at several locations in the taskfiles.

It might be that a project that is not a Go module contains an accessory Go module in a subfolder. In this case, the use
of a taskfile variable will allow that path to be defined in one place, avoiding the need to override the hardcoded
incorrect location whenever a Go module-related task is run manually.
@per1234 per1234 added topic: code Related to content of the project itself type: enhancement Proposed improvement labels Aug 15, 2021
@per1234 per1234 requested a review from umbynos August 15, 2021 04:31
@per1234 per1234 merged commit 817245d into arduino:main Aug 16, 2021
@per1234 per1234 deleted the default-go-module-path-var branch August 16, 2021 10:10
@per1234 per1234 self-assigned this Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants