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

FS-3797 #206

Merged
merged 8 commits into from
Nov 30, 2023
Merged

FS-3797 #206

merged 8 commits into from
Nov 30, 2023

Conversation

robk-dluhc
Copy link
Contributor

Correct workflow order.

  • 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.

Update LGTM too

@robk-dluhc robk-dluhc merged commit ea9a235 into main Nov 30, 2023
26 of 29 checks passed
@robk-dluhc robk-dluhc deleted the FS-3797 branch November 30, 2023 11:25
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