Skip to content

Commit 7212057

Browse files
committed
MB-65635 Fix pre-push hook
* Don't try to lint cbbackup &co - they've been removed * Define `columnar` in `Subcommand` to silence lint Change-Id: I0a893aa3de541920f9889f5c9da93ebc989627ee Reviewed-on: https://review.couchbase.org/c/couchbase-cli/+/224480 Tested-by: Build Bot <build@couchbase.com> Reviewed-by: Safian Ali <safian.ali@couchbase.com>
1 parent 0d298c9 commit 7212057

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

.githooks/pre-push

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,5 @@ done
1515
# run multiple linters over the codebase before allowing allowing a push (these are the same linters that the build bot
1616
# runs), therefore, this should mean less wasted time spent fixing mistakes picked up when it's least convenient to fix
1717
# them.
18-
pylint --errors-only --disable=import-error cbbackup cbbackupwrapper cblogredaction cbrecovery cbrestore \
19-
cbrestorewrapper cbtransfer cbworkloadgen couchbase-cli pump*.py
18+
pylint --errors-only --disable=import-error cblogredaction cbrecovery cbtransfer cbworkloadgen couchbase-cli pump*.py
2019
pylint --errors-only --disable=import-error,unused-import --disable C,R cbmgr.py cluster_manager.py

cbmgr.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -747,6 +747,7 @@ def __init__(self, deprecate_username=False, deprecate_password=False, cluster_d
747747
# Filled by the decorators
748748
self.rest = None
749749
self.enterprise = None
750+
self.columnar = None
750751

751752
self.parser = CliParser(formatter_class=CLIHelpFormatter, add_help=False, allow_abbrev=False)
752753
group = self.parser.add_argument_group("Cluster options")

0 commit comments

Comments
 (0)