Skip to content
This repository was archived by the owner on May 13, 2026. It is now read-only.

chore: bump eslint-config-moneyforward from 1.0.0 to 2.0.0#280

Merged
horaotoko merged 2 commits into
mainfrom
dependabot/npm_and_yarn/eslint-config-moneyforward-2.0.0
Jan 18, 2023
Merged

chore: bump eslint-config-moneyforward from 1.0.0 to 2.0.0#280
horaotoko merged 2 commits into
mainfrom
dependabot/npm_and_yarn/eslint-config-moneyforward-2.0.0

Conversation

@dependabot
Copy link
Copy Markdown
Contributor

@dependabot dependabot Bot commented on behalf of github Jan 18, 2023

Bumps eslint-config-moneyforward from 1.0.0 to 2.0.0.

Release notes

Sourced from eslint-config-moneyforward's releases.

Release 2.0.0

  • Update @​typescript-eslint/parser requirement from ^5.48.1 to ^5.48.2 (2d682ec)
  • Update @​typescript-eslint/eslint-plugin requirement (ee4429c)
  • Update eslint-plugin-react requirement from ^7.32.0 to ^7.32.1 (baf3290)
  • Update prettier requirement from ^2.8.2 to ^2.8.3 (c3252ef)
  • Update eslint requirement from ^8.31.0 to ^8.32.0 (19cfc8e)
  • Update eslint-plugin-react requirement from ^7.31.11 to ^7.32.0 (c7cab27)
  • Update eslint-plugin-testing-library requirement from ^4.6.0 || ^5.0.0 to ^5.9.1 (0e4dc59)
  • Update eslint-plugin-jest requirement from ^27.2.0 to ^27.2.1 (5fd34b5)
  • Update eslint-plugin-react-hooks requirement from ^4.2.0 to ^4.6.0 (032c00b)
  • Update @​typescript-eslint/parser requirement from ^4.28.0 || ^5.0.0 to ^5.48.1 (759967f)
  • Bump release-it from 14.14.3 to 15.6.0 (051698e)
  • Update @​typescript-eslint/eslint-plugin requirement from ^4.28.0 || ^5.0.0 to ^5.48.1 (5cc0825)
  • Update eslint-plugin-react requirement from ^7.24.0 to ^7.31.11 (d836838)
  • Bump jest from 27.5.1 to 29.3.1 (5dfde18)
  • Update prettier requirement from ^2.8.1 to ^2.8.2 (cf80a7f)
  • Update eslint-config-prettier requirement from ^8.3.0 to ^8.6.0 (7feb4ed)
  • Merge pull request #81 from moneyforward/finish-support-node-v12 (bbb6bc6)
  • update: finish support for node v12 (3f0ab33)
  • Bump eslint-plugin-jest from 25.7.0 to 27.2.0 (91a8f37)
  • Update prettier requirement from ^2.3.2 to ^2.8.1 (172762d)
  • Bump eslint-plugin-jest-dom from 3.9.4 to 4.0.3 (53cf139)
  • Bump eslint from 7.32.0 to 8.31.0 (118524e)
  • Merge pull request #58 from moneyforward/add-support-for-nodejs-18 (cd4de6c)
  • add support for Node.js v18 (d7a5063)
  • Merge pull request #23 from horaotoko/horaotoko/patch/dependabot (c184a85)
  • lint fix (3607ba4)
  • dependabot init (2a3bf2e)

Release 1.0.1

  • Merge pull request #20 from moneyforward/chore/use-eslint8 (53a11bf)
  • chore(actions): check result with eslint8 (4136b3b)
  • chore(deps): relax dependencies for eslint8 (c0b4ecd)
  • Merge pull request #19 from moneyforward/chore/eslint-matrix (15e30aa)
  • chore(actions): ignore error on eslint v8 (ef1382d)
  • chore(actions): run eslint v8 (8c23761)
  • Merge pull request #15 from moneyforward/add-support-for-nodejs-12-14 (640139f)
  • Add support for nodejs v12/v14 (77446ab)
Commits
  • baffed6 Release 2.0.0
  • 2d682ec Update @​typescript-eslint/parser requirement from ^5.48.1 to ^5.48.2
  • ee4429c Update @​typescript-eslint/eslint-plugin requirement
  • baf3290 Update eslint-plugin-react requirement from ^7.32.0 to ^7.32.1
  • c3252ef Update prettier requirement from ^2.8.2 to ^2.8.3
  • 19cfc8e Update eslint requirement from ^8.31.0 to ^8.32.0
  • c7cab27 Update eslint-plugin-react requirement from ^7.31.11 to ^7.32.0
  • 0e4dc59 Update eslint-plugin-testing-library requirement from ^4.6.0 || ^5.0.0 to ^5.9.1
  • 5fd34b5 Update eslint-plugin-jest requirement from ^27.2.0 to ^27.2.1
  • 032c00b Update eslint-plugin-react-hooks requirement from ^4.2.0 to ^4.6.0
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [eslint-config-moneyforward](https://github.com/moneyforward/eslint-config-moneyforward) from 1.0.0 to 2.0.0.
- [Release notes](https://github.com/moneyforward/eslint-config-moneyforward/releases)
- [Commits](moneyforward/frontend-tools@1.0.0...2.0.0)

---
updated-dependencies:
- dependency-name: eslint-config-moneyforward
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot requested a review from a team as a code owner January 18, 2023 03:43
@dependabot dependabot Bot requested review from horaotoko and removed request for a team January 18, 2023 03:43
@dependabot dependabot Bot added the dependencies Pull requests that update a dependency file label Jan 18, 2023
Comment thread package.json
"@types/styled-components": "^5.1.26",
"@typescript-eslint/eslint-plugin": "^4.33.0",
"@typescript-eslint/parser": "^4.33.0",
"@typescript-eslint/eslint-plugin": "^5.48.2",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These @typescript-eslint are conflicted with eslint-config-moneyforward

Comment thread package.json
"eslint": "^7.32.0",
"eslint-config-moneyforward": "^1.0.0",
"eslint-config-moneyforward": "^2.0.0",
"eslint-plugin-jest": "^27.2.1",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This eslint-plugin-jest is required for eslint-config-moneyforward

fireEvent.click(screen.getByRole('button', { name: '閉じる' }));
});

it('props.onClickClose is called', () => {
Copy link
Copy Markdown
Member

@horaotoko horaotoko Jan 18, 2023

Choose a reason for hiding this comment

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

This test was redundant.
I bundled together in a case.

@horaotoko horaotoko requested a review from kimromi January 18, 2023 05:04
@horaotoko
Copy link
Copy Markdown
Member

@kimromi
Code review plz 🙏

Copy link
Copy Markdown
Contributor

@kimromi kimromi left a comment

Choose a reason for hiding this comment

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

@horaotoko thanks:pray:

render(<Backdrop open onClick={onClick} data-testid="testid" />);
fireEvent.click(screen.getByTestId('testid'));
expect(onClick).toBeCalledTimes(1);
expect(onClick).toHaveBeenCalledTimes(1);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@horaotoko horaotoko merged commit 6a747cc into main Jan 18, 2023
@horaotoko horaotoko deleted the dependabot/npm_and_yarn/eslint-config-moneyforward-2.0.0 branch January 18, 2023 08:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants