Skip to content

Verify info spec refactor#7911

Merged
soniaconnolly merged 3 commits intomainfrom
sonia-luis-verify-info-spec-refactor
Mar 1, 2023
Merged

Verify info spec refactor#7911
soniaconnolly merged 3 commits intomainfrom
sonia-luis-verify-info-spec-refactor

Conversation

@soniaconnolly
Copy link
Contributor

Undo some of the spec changes from #7847.

Co-authored-by: Luis H. Matos ThatSpaceGuy@users.noreply.github.com

soniaconnolly and others added 2 commits March 1, 2023 12:30
And take out some indirection in verify_info_step_spec feature spec.

Co-authored-by: Luis H. Matos <ThatSpaceGuy@users.noreply.github.com>
@soniaconnolly soniaconnolly marked this pull request as ready for review March 1, 2023 20:47
@soniaconnolly soniaconnolly requested review from a team and ThatSpaceGuy March 1, 2023 20:47
Copy link
Contributor

@jmax-gsa jmax-gsa left a comment

Choose a reason for hiding this comment

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

LGTM

Nice cleanup.

@soniaconnolly soniaconnolly merged commit 09b799f into main Mar 1, 2023
@soniaconnolly soniaconnolly deleted the sonia-luis-verify-info-spec-refactor branch March 1, 2023 21:20
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