core
core copied to clipboard
Merge fixes for issues 749388
Pull Request: Merge Fixes from Multiple Branches
Description
This pull request merges changes from four branches (devin/fix-issues/749388/5
, devin/fix-issues/749388/2
, devin/fix-issues/749388/1
, and devin/fix-issues/749388/4
) into a new branch named devin/fix-issues/749388/merge
in the core
repository. The following tasks were completed as part of this merge:
- Merged changes from the specified branches into the
devin/fix-issues/749388/merge
branch. - Resolved merge conflicts to retain all fixes from the merged branches.
- Enabled Corepack and installed dependencies using Yarn.
- Addressed initial linting issues and ran
yarn lint --fix
to automatically fix linting errors. - Resolved test suite failures related to type exports, specifically for
LogType
,SigningMethod
, andSigningStage
. - Manually fixed linting errors in
packages/notification-controller/src/index.ts
to comply with formatting rules. - Re-ran the linter to ensure all linting issues were resolved.
- Re-ran the test suites to verify that no new issues were introduced.
- Fixed lint errors in
index.test.ts
files by removing export statements.
Changes
- Merged changes from branches
devin/fix-issues/749388/5
,devin/fix-issues/749388/2
,devin/fix-issues/749388/1
, anddevin/fix-issues/749388/4
. - Fixed linting issues in
packages/logging-controller/src/index.ts
andpackages/notification-controller/src/index.ts
. - Resolved test suite failures related to type exports.
- Fixed lint errors in
index.test.ts
files by removing export statements.
Checklist
- [x] Merged changes from specified branches.
- [x] Resolved merge conflicts.
- [x] Enabled Corepack and installed dependencies.
- [x] Addressed linting issues and ran
yarn lint --fix
. - [x] Resolved test suite failures.
- [x] Manually fixed linting errors in
packages/notification-controller/src/index.ts
. - [x] Re-ran linter and test suites.
- [x] Fixed lint errors in
index.test.ts
files by removing export statements.
Link to Devin run
This Devin run was requested by Devin.
Please review the changes and provide feedback.