Skip to content

Commit 585b359

Browse files
committed
cpplint strictly requires Python 2
Using Python 3 (as is the default GitHub's Ubuntu 20.04) requires merging changes from https://github.com/cpplint/cpplint, which is a larger undertaking.
1 parent e3aabfa commit 585b359

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

.github/workflows/pull-request-checks.yaml

+2-1
Original file line numberDiff line numberDiff line change
@@ -364,7 +364,8 @@ jobs:
364364
# user input
365365
DEBIAN_FRONTEND: noninteractive
366366
run: |
367-
pip install unidiff
367+
sudo apt-get update
368+
sudo apt-get install --no-install-recommends -yq python3-unidiff
368369
- name: Check updated lines of code meet linting standards
369370
env:
370371
BASE_BRANCH: ${{ github.base_ref }}

scripts/cpplint.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
#!/usr/bin/env python
1+
#!/usr/bin/env python2
22
# -*- coding: utf-8 -*-
33
#
44
# Copyright (c) 2009 Google Inc. All rights reserved.

scripts/diff_to_added_lines.py

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
1-
#!/usr/bin/env python
1+
#!/usr/bin/env python3
22

3-
from __future__ import print_function
43
import sys
54

65
def eprint(*args, **kwargs):

0 commit comments

Comments
 (0)