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

FS-3797 #161

Merged
merged 6 commits into from
Nov 30, 2023
Merged

FS-3797 #161

merged 6 commits into from
Nov 30, 2023

Conversation

robk-dluhc
Copy link
Contributor

Workflow changes following review.

  • Unit tests and other appropriate tests added or updated
  • README and other documentation has been updated / added (if needed)
  • Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

Copy link
Contributor

@aaronwilliamsv1 aaronwilliamsv1 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@aaronwilliamsv1 aaronwilliamsv1 left a comment

Choose a reason for hiding this comment

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

LGTM v2

Copy link
Contributor

@aaronwilliamsv1 aaronwilliamsv1 left a comment

Choose a reason for hiding this comment

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

LGTM v3.0

@robk-dluhc robk-dluhc merged commit a92dfcb into main Nov 30, 2023
22 of 23 checks passed
@robk-dluhc robk-dluhc deleted the FS-3797 branch November 30, 2023 12:21
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