8000 Add go.work file to root by Felttrip · Pull Request #1410 · temporalio/ui · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add go.work file to root #1410

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 5 commits into from
Jun 22, 2023
Merged

Add go.work file to root #1410

merged 5 commits into from
Jun 22, 2023

Conversation

Felttrip
Copy link
Contributor
@Felttrip Felttrip commented Jun 5, 2023

Description & motivation 💭

This PR adds a go.work file to the root of the project. This allows gopls and the tools that leverage it, like VSCodes go extension, to be aware that there is a go module in this project not at the root of the project. When gopls is aware of the module functionality like autocomplete and goto work as expected when opening the project from root without the project living in your GOPATH.

Ref
https://go.dev/ref/mod#workspaces
https://github.com/golang/tools/blob/master/gopls/doc/workspace.md

Screenshots (if applicable) 📸

Screen.Recording.2023-06-05.at.1.24.02.PM.mov

Design Considerations 🎨

N/A

Testing 🧪

Manual validation in VSCode

How was this tested 👻

  • Manual testing
  • E2E tests added
  • Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

Pull the branch, validate that your go extension of choice works.

Checklists

Draft Checklist

Merge Checklist

Issue(s) closed

Docs

Any docs updates needed?

@Felttrip Felttrip requested a review from a team as a code owner June 5, 2023 19:26
@Felttrip Felttrip requested review from GiantRobots and removed request for a team June 5, 2023 19:26
@vercel
Copy link
vercel bot commented Jun 5, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
holocene ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 22, 2023 3:54pm

@CLAassistant
Copy link
CLAassistant commented Jun 5, 2023

CLA assistant check
All committers have signed the CLA.

@rossedfort rossedfort requested review from robholland, feedmeapples and rossedfort and removed request for GiantRobots June 5, 2023 19:30
@feedmeapples
Copy link
Contributor
feedmeapples commented Jun 14, 2023

keeping this open until fixing the e2e tests in the PR. Haven't freed up some time to do this yet

@rossedfort
Copy link
Contributor

keeping this open until fixing the e2e tests in the PR. Haven't freed up some time to do this yet

No worries @feedmeapples, we are currently migrating the e2e test utils away from Go, and have some flakey tests. This should be resolved once we merge main into this branch after merging #1414

@rossedfort rossedfort merged commit a04ebf5 into temporalio:main Jun 22, 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.

4 participants
0