Remove "use Workflow.getLogger" comment from SagaWorkflow example
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 48/100
- Issue type
- Documentation
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- php
- Domain
- documentation
Research direction
Search the SagaWorkflow example for the quoted "use Workflow.getLogger" comment and read the surrounding example code. Remove the contradictory comment, then verify that the example no longer tells users to call a method that the issue says does not exist.
Written by the indexing model from the issue text.
Description
There is a comment in SagaWorkflow example which contradicts SDK code:
In production code please always use Workflow.getLogger to log messages in workflow code.
There is no Workflow::getLogger method.
- Dominant language
- PHP
- Stars
- 125
- Forks
- 46
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from temporalio/samples-php
-
enhancement
Difficulty 4/5 3-5 days Newbie friendliness 25/100
temporalio/samples-php#68 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 30/100
temporalio/samples-php#65 · 2 comments ·
-
enhancement
Difficulty 4/5 3-5 days Newbie friendliness 25/100
temporalio/samples-php#47 ·
-
Docker image error Openbug
temporalio/samples-php#42 · 2 comments · 1 assignee ·
-
enhancement
Difficulty 3/5 1-2 days Newbie friendliness 35/100
temporalio/samples-php#39 · 1 comment · 1 reaction ·
All issues in temporalio/samples-php
Similar issues
-
Solved site promotion gate fails on runner PHP patch drift (expects 8.2.33, runner installs 8.2.34) Open
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Automattic/blocks-engine#2161 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Automattic/static-site-importer#1824 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
0. Needs triage bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
0. Needs triage 35-feedback bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100