Skip to content

Change fsm_ to _flow_state_machine to eliminate the acronym#7076

Merged
gangelo merged 2 commits intomainfrom
eliminate-acronyms-and-abbreviations
Oct 5, 2022
Merged

Change fsm_ to _flow_state_machine to eliminate the acronym#7076
gangelo merged 2 commits intomainfrom
eliminate-acronyms-and-abbreviations

Conversation

@gangelo
Copy link
Contributor

@gangelo gangelo commented Oct 3, 2022

🎫 Ticket

N/A

🛠 Summary of changes

Change fsm_ to _flow_state_machine to eliminate the cryptic acronyms.

📜 Testing Plan

  • All existing tests should pass.

👀 Screenshots

N/A

🚀 Notes for Deployment

N/A

@gangelo gangelo self-assigned this Oct 3, 2022
@gangelo gangelo requested a review from jmhooper October 3, 2022 17:45
@gangelo gangelo force-pushed the eliminate-acronyms-and-abbreviations branch from 3dbca52 to 57fcf91 Compare October 3, 2022 18:02
@gangelo gangelo marked this pull request as ready for review October 3, 2022 18:04
To initialize_flow_state_machine
@gangelo gangelo requested a review from aduth October 5, 2022 12:51
@gangelo gangelo changed the title Change fsm_ to flow_state_machine_ to eliminate the acronym Change fsm_ to _flow_state_machine to eliminate the acronym Oct 5, 2022
Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

👍

@gangelo gangelo merged commit f8d6bc6 into main Oct 5, 2022
@gangelo gangelo deleted the eliminate-acronyms-and-abbreviations branch October 5, 2022 15:13
jskinne3 pushed a commit that referenced this pull request Oct 12, 2022
* Change fsm_ to floe_state_machine_ to eliminate the acronym

[skip changelog]

* Rename flow_state_machine_initialize

To initialize_flow_state_machine
@rnagilla-gsa rnagilla-gsa added the inherited proofing Pull Requests for the Inherited Proofing feature label Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

inherited proofing Pull Requests for the Inherited Proofing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants