Skip to content

Conversation

@ansidev
Copy link
Contributor

@ansidev ansidev commented Aug 30, 2025

What changed?

  • Update Golang Hello World tutorial: register workflow and activity using worker instance due to the deprecated global functions.
  • Convert 4-spaces indentation to tab because of Go coding styles.

Why?
Update Golang Hello World tutorial.

How did you test it?
local tests

}
// add to the function startWorker before calling worker.Start()

// if err != nil {
Copy link
Member

Choose a reason for hiding this comment

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

mind removing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@davidporter-id-au I added those lines intentionally. Btw, I have removed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@davidporter-id-au Are there any concerns from you about this PR?

@ansidev ansidev force-pushed the fix-golang-hello-world-tutorial branch from b192810 to 932a44c Compare September 1, 2025 08:06
@ansidev
Copy link
Contributor Author

ansidev commented Sep 3, 2025

@shijiesheng Would you mind merging this PR? I do not have the permissions to do that.

Copy link
Member

@davidporter-id-au davidporter-id-au left a comment

Choose a reason for hiding this comment

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

no concerns, sorry if I misinterpreted the if/err != nil thing earlier, it looked at a glance like accidentally commented code

@ansidev
Copy link
Contributor Author

ansidev commented Sep 3, 2025

@davidporter-id-au Could you please merge the PR? I do not have the permissions to do that.

@Bueller87 Bueller87 merged commit db405be into cadence-workflow:master Sep 4, 2025
1 check passed
@ansidev ansidev deleted the fix-golang-hello-world-tutorial branch September 4, 2025 01:50
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