Skip to content

Cb 6263 lefthook#3291

Merged
Wroud merged 15 commits intodevelfrom
CB-6263-lefthook
Mar 17, 2025
Merged

Cb 6263 lefthook#3291
Wroud merged 15 commits intodevelfrom
CB-6263-lefthook

Conversation

@devnaumov
Copy link
Copy Markdown
Member

No description provided.

Comment thread .license-checker.txt Outdated
Comment on lines +1 to +7
/*
* CloudBeaver - Cloud Database Manager
* Copyright (C) 2020-2025 DBeaver Corp and others
*
* Licensed under the Apache License, Version 2.0.
* you may not use this file except in compliance with the License.
*/ No newline at end of file
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.

lets have license 1 entry point for vscode snippet and license checking via git hooks

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.

agree, it will be strange to support both of them

Comment thread webapp/package.json
Comment thread .license-checker.txt Outdated
Comment on lines +1 to +7
/*
* CloudBeaver - Cloud Database Manager
* Copyright (C) 2020-2025 DBeaver Corp and others
*
* Licensed under the Apache License, Version 2.0.
* you may not use this file except in compliance with the License.
*/ No newline at end of file
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.

agree, it will be strange to support both of them

Comment thread webapp/.husky/_/pre-commit Outdated
@sergeyteleshev
Copy link
Copy Markdown
Contributor

image
image

please fix the error so it more readable

Comment thread webapp/packages/core-cli/package.json Outdated
Comment thread webapp/licensifyrc.yml
Comment thread lefthook.yml Outdated
Wroud
Wroud previously approved these changes Mar 13, 2025
sergeyteleshev
sergeyteleshev previously approved these changes Mar 13, 2025
@devnaumov devnaumov dismissed stale reviews from Wroud and sergeyteleshev via 2c19bef March 13, 2025 14:01
@Wroud Wroud merged commit 34ff5a3 into devel Mar 17, 2025
4 checks passed
@Wroud Wroud deleted the CB-6263-lefthook branch March 17, 2025 08:07
kkapper pushed a commit to PlaidCloud/cloudbeaver that referenced this pull request Apr 18, 2025
* CB-6263 migrate to lefthook

* CB-6263 remove old pre-commit

* CB-6263 specify how to execute lefthook

* CB-6263 add config file

* CB-6263 get license from configuration file

* CB-6263 remove old script

* CB-6263 remove old license

* CB-6263 add locales check

* CB-6283 fix vscode commit msg

* CB-6263 filter staged files on lib level

* CB-6263 do not run hooks sync in CI

* CB-6263 change script

* CB-6263 revert postinstall

---------

Co-authored-by: mr-anton-t <42037741+mr-anton-t@users.noreply.github.com>
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.

4 participants