Skip to content

[CCFPCM-355] Eslint/Autofix#45

Merged
chelsea-EYDS merged 2 commits intomainfrom
CCFPCM-355
Feb 7, 2023
Merged

[CCFPCM-355] Eslint/Autofix#45
chelsea-EYDS merged 2 commits intomainfrom
CCFPCM-355

Conversation

@chelsea-EYDS
Copy link
Contributor

@chelsea-EYDS chelsea-EYDS commented Feb 7, 2023

CCFPCM-355

Objective:

Update ESLint Auto-fix Rules

What has changed:

  • eslintrc
  • autofix on imports

Tasks:

  • Updated ESlint rules to use automated import sorting and automated removal of unused imports.

notes:
This will run with the pre-commit hook, or, you can run yarn run eslint-nibble from the project root directory to autofix outside of a commit

Copy link

@jwarrenbc jwarrenbc left a comment

Choose a reason for hiding this comment

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

Reviewed & tested build

@chelsea-EYDS chelsea-EYDS merged commit 97b10e9 into main Feb 7, 2023
chelsea-EYDS added a commit that referenced this pull request Feb 8, 2023
* [Master Data] Update locations with seed migration, fix types

* [REFACTOR] Match & kill (#43)

* updated recon lambda
* update deposit match
* Fix entities, use typeorm to get relations, update to strict time based matching
* update save pos results, cash updates wip.
* TYPE! amounts, fix up cash deposit with 1:1 look around match
* refine pos matching with ordering and eager kill
* summary reporting update.
* Add eslint check
* make pr checks better
* yarn build pr check

---------

Co-authored-by: Vyas <67800715+vyasworks@users.noreply.github.com>

* [CCFPCM-355] Eslint/Autofix (#45)

* [CCFPCM-355] Eslint/Autofix

* [CCFPCM-355] Eslint/Autofix

* [CCFPCM-337]: time field parsing

* [CCFPCM-337]: time field parsing

---------

Co-authored-by: Vyas <67800715+vyasworks@users.noreply.github.com>
@chelsea-EYDS chelsea-EYDS deleted the CCFPCM-355 branch March 22, 2023 22:59
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.

3 participants