Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

doc: expand comments in WorkflowSystemTask #2456

Merged
merged 1 commit into from
Sep 13, 2021

Conversation

Jiehong
Copy link
Contributor

@Jiehong Jiehong commented Sep 11, 2021

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Other (please describe):

Code comment update. (due to my better understand of the matter, started in #2450)

Just to make it clearer what "start" and "execute" mean when someone wants to extend WorkflowSystemTask.

Also, any reason why WorkflowSystemTask is an abstract class, instead of an interface?

@kishorebanala kishorebanala merged commit 2d25ae8 into Netflix:main Sep 13, 2021
@github-actions
Copy link
Contributor

Unit Test Results

   152 files  ±0     152 suites  ±0   8m 7s ⏱️ ±0s
1 115 tests ±0  1 115 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 2d25ae8. ± Comparison against base commit 2d25ae8.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants