Skip to content
This repository has been archived by the owner on Jul 23, 2022. It is now read-only.

Apply Prettier #1202

Merged
merged 9 commits into from
Dec 20, 2021
Merged

Apply Prettier #1202

merged 9 commits into from
Dec 20, 2021

Conversation

Hinton
Copy link
Member

@Hinton Hinton commented Dec 20, 2021

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Sync existing branches:

  1. Check out your local Branch
  2. Run git merge
  3. Resolve any merge conflicts, commit.
  4. Run npm prettier
  5. Commit
  6. Run git merge -Xours
  7. Push

Testing requirements

N/A Should be no functional changes

Before you submit

  • I have checked for linting errors (npm run lint) (required)
  • I have added unit tests where it makes sense to do so (encouraged but not required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

@Hinton
Copy link
Member Author

Hinton commented Dec 20, 2021

e437976 and 413ab3c looks good.

@Hinton Hinton marked this pull request as ready for review December 20, 2021 09:35
@Hinton Hinton requested a review from djsmith85 December 20, 2021 09:36
djsmith85
djsmith85 previously approved these changes Dec 20, 2021
Copy link
Contributor

@djsmith85 djsmith85 left a comment

Choose a reason for hiding this comment

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

Changes are looking good.

@Hinton Hinton merged commit 521feae into master Dec 20, 2021
@Hinton Hinton deleted the feature/apply-prettier branch December 20, 2021 14:47
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