New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build(deps) npm update + shadow-bump of commitizen #218
Conversation
Pull Request Report PR Title ❌ Title should follow the conventional commit spec: (optional scope): Example: feat(cli): add new command |
({ | ||
getAllOrgsUserHasAccessTo: mockGetOrgs, | ||
} as unknown) as AuthenticatedClient) | ||
} as unknown as AuthenticatedClient) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prettier has spoken.
@@ -11,7 +11,7 @@ describe('hooks:prerun', () => { | |||
const mockSet = jest.fn(); | |||
|
|||
mockConfig.mockImplementation( | |||
() => (({get: mockGet, set: mockSet} as unknown) as Config) | |||
() => ({get: mockGet, set: mockSet} as unknown as Config) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prettier has spoken.
({ | ||
set: mockSet, | ||
} as unknown) as Config) | ||
} as unknown as Config) | ||
); | ||
|
||
mockedClient.mockImplementation( | ||
() => | ||
(({ | ||
({ | ||
getUserHasAccessToOrg: mockGetHasAccessToOrg, | ||
} as unknown) as AuthenticatedClient) | ||
} as unknown as AuthenticatedClient) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prettier has spoken.
({ | ||
get: mockGet, | ||
set: mockSet, | ||
} as unknown) as Config) | ||
} as unknown as Config) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prettier has spoken.
({ | ||
get: mockConfigGet, | ||
set: mockConfigSet, | ||
} as unknown) as Config) | ||
} as unknown as Config) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prettier has spoken.
({ | ||
getAllOrgsUserHasAccessTo: mockListOrgs, | ||
getUserHasAccessToOrg: mockGetHasAccessToOrg, | ||
} as unknown) as AuthenticatedClient) | ||
} as unknown as AuthenticatedClient) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prettier has spoken.
779ef45
to
e9b8fc3
Compare
Issue: https://github.com/coveo/cli/security/dependabot/package-lock.json/merge/open
Diagnostic:
Resolution: commitizen/cz-cli#824
On our end, we just 'update' the sub dep (it's janky: install the dep version you want, then uninstall it. The pkg-lock tho will keep the version you installed).
This makes so we have our dependency tree updated and don't have to wait for
cz-conventional-changelog
to be bumped.Also, bonus round: I ran an
npm update
to dust off our deps.Edit: Bonus round 2. Updated to prettier 2.3.0
https://prettier.io/blog/2021/05/09/2.3.0.html
Some minor changes, reapplied prettier to the project to keep it tidy
CDX-227