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

fix: Remove references to cloning weather sample with gonew #139

Merged
merged 1 commit into from
Oct 21, 2023

Conversation

ikawaha
Copy link
Contributor

@ikawaha ikawaha commented Oct 21, 2023

issue:#131

I apologize for the oversight in the previous PR d9ffbdc. It turns out that the weather example from the clue repository cannot be cloned using the gonew command. The reason is that the weather example consists of multiple services and is not structured as a single module. Since gonew is designed to clone a single module as a template, it cannot target the weather example.

@raphael
Copy link
Member

raphael commented Oct 21, 2023

OK makes sense. Maybe we should make the example its own module and define a go workspace at the top level of the repo. I'll work on that.

@raphael raphael merged commit aab0827 into goadesign:master Oct 21, 2023
1 check passed
@ikawaha ikawaha deleted the fix/readme_20231021 branch October 21, 2023 04:58
@raphael
Copy link
Member

raphael commented Oct 21, 2023

done: goadesign/clue#312

ikawaha added a commit to ikawaha/examples that referenced this pull request Oct 25, 2023
raphael pushed a commit that referenced this pull request Oct 26, 2023
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.

None yet

2 participants