Skip to content
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

Propose Change: Default Module Name to Project Name #3277

Closed
Owbird opened this issue Feb 25, 2024 · 3 comments · Fixed by #3303
Closed

Propose Change: Default Module Name to Project Name #3277

Owbird opened this issue Feb 25, 2024 · 3 comments · Fixed by #3303
Labels
Enhancement New feature or request

Comments

@Owbird
Copy link

Owbird commented Feb 25, 2024

Is your feature request related to a problem? Please describe.

No response

Describe the solution you'd like

The solution I propose is to modify the Go Wails project templates to automatically generate a go.mod file with at least the project name as the module name, instead of the default placeholder "changeme".

Describe alternatives you've considered

No response

Additional context

No response

@Owbird Owbird added the Enhancement New feature or request label Feb 25, 2024
@leaanthony
Copy link
Member

Feel free to create a PR that does this. I would suggest having it as a CLI parameter.

@PylotLight
Copy link
Contributor

This could also benefit templates that include project imports so the imports are templated using the provided arg.
E.g my htmx template that imports components using the module name could benefit from the templated name.

@Twacqwq
Copy link
Contributor

Twacqwq commented Mar 11, 2024

Hi @leaanthony,

I tried adding a CLI parameter to the init command that changes the default module name to the project name.
Could you please take a moment to review it when you have the time? thx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants