Skip to content

V4 general refactoring#3

Open
nikolai-laevskii wants to merge 12 commits intomainfrom
v4-general-refactoring
Open

V4 general refactoring#3
nikolai-laevskii wants to merge 12 commits intomainfrom
v4-general-refactoring

Conversation

@nikolai-laevskii
Copy link

@nikolai-laevskii nikolai-laevskii commented Aug 2, 2023

Description:

  • codestyle overhaul: eliminated too much nesting to make everything more readable
  • wherever code with distinct purpose could be moved to a new function it was done
  • smaller files for easier change reviews
  • iteration over changed pull requests has been moved to its own level of abstraction

Related issue:
https://github.com/actions/setup-actions-team/issues/20

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@nikolai-laevskii nikolai-laevskii self-assigned this Aug 2, 2023
@dsame dsame self-assigned this Aug 21, 2023
MaksimZhukov pushed a commit that referenced this pull request Sep 4, 2023
@joshdales
Merge pull request #3 from joshdales/new-config-structure
8943ca2
dependabot bot added 5 commits September 13, 2023 10:14
@dependabot
Bump @typescript-eslint/eslint-plugin from 6.6.0 to 6.7.0 (actions#676)
725d4ca 
Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 6.6.0 to 6.7.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v6.7.0/packages/eslint-plugin)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @typescript-eslint/parser from 6.6.0 to 6.7.0 (actions#677)
6c834e4 
Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 6.6.0 to 6.7.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v6.7.0/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump eslint-plugin-jest from 27.2.3 to 27.4.0 (actions#678)
c533686 
Bumps [eslint-plugin-jest](https://github.com/jest-community/eslint-plugin-jest) from 27.2.3 to 27.4.0.
- [Release notes](https://github.com/jest-community/eslint-plugin-jest/releases)
- [Changelog](https://github.com/jest-community/eslint-plugin-jest/blob/main/CHANGELOG.md)
- [Commits](jest-community/eslint-plugin-jest@v27.2.3...v27.4.0)

---
updated-dependencies:
- dependency-name: eslint-plugin-jest
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @types/js-yaml from 4.0.5 to 4.0.6 (actions#679)
519694e 
Bumps [@types/js-yaml](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/js-yaml) from 4.0.5 to 4.0.6.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/js-yaml)

---
updated-dependencies:
- dependency-name: "@types/js-yaml"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
Bump @typescript-eslint/parser from 6.7.0 to 6.7.2 (actions#682)
2b2e7d0 
Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 6.7.0 to 6.7.2.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v6.7.2/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
dependabot bot and others added 3 commits October 4, 2023 09:02
@dependabot
Bump @typescript-eslint/eslint-plugin from 6.7.0 to 6.7.4 (actions#690)
8207c14 
Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 6.7.0 to 6.7.4.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v6.7.4/packages/eslint-plugin)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@nikolai-laevskii
Use lodash's isEqual
df65a71
@nikolai-laevskii
Merge remote-tracking branch 'origin' into v4-general-refactoring
dd1ca3d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MaksimZhukov MaksimZhukov MaksimZhukov approved these changes

+2 more reviewers

@IvanZosimov IvanZosimov IvanZosimov approved these changes

@marko-zivic-93 marko-zivic-93 marko-zivic-93 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@dsame dsame

@nikolai-laevskii nikolai-laevskii

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@nikolai-laevskii @MaksimZhukov @IvanZosimov @marko-zivic-93 @dsame