Skip to content

Normalize naming prevStep -> step#17726

Merged
wmitsuda merged 1 commit into
mainfrom
wmitsuda/clarify-domain-inner-workings
Oct 31, 2025
Merged

Normalize naming prevStep -> step#17726
wmitsuda merged 1 commit into
mainfrom
wmitsuda/clarify-domain-inner-workings

Conversation

@wmitsuda
Copy link
Copy Markdown
Member

Small rename refactoring to clarify meaning in the code:

  • The concept of "prev step" is only used in SharedDomains as GetLatest() is used as prev value of next mutation
  • Inside GetLatest() chain of calls it is actually just the step whose that latest mutation occurred, "prev step" sounds strange and confusing when reading the code
  • Most of GetLatest() chain of calls are already using only "step" as the named variable, so this PR just completes the standardization of naming.

@wmitsuda wmitsuda enabled auto-merge (squash) October 31, 2025 08:31
@wmitsuda wmitsuda merged commit ac5c9e8 into main Oct 31, 2025
17 checks passed
@wmitsuda wmitsuda deleted the wmitsuda/clarify-domain-inner-workings branch October 31, 2025 09:35
mh0lt pushed a commit that referenced this pull request Nov 4, 2025
Small rename refactoring to clarify meaning in the code:

- The concept of "prev step" is only used in SharedDomains as
GetLatest() is used as prev value of next mutation
- Inside GetLatest() chain of calls it is actually just the step whose
that latest mutation occurred, "prev step" sounds strange and confusing
when reading the code
- Most of GetLatest() chain of calls are already using only "step" as
the named variable, so this PR just completes the standardization of
naming.
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.

2 participants