Skip to content

Commit

Permalink
Use 'python3' instead of 'python' in scripts (#25499)
Browse files Browse the repository at this point in the history
These scripts are run in the *host*, not a container, so they should
reference 'python3' explicitly to make sure they get a useful Python
version. According to PEP 394, 'python' could refer to Python 2 in a
global environment, so 'python3' should be preferred for scripts.
  • Loading branch information
uranusjr authored Aug 3, 2022
1 parent 298be50 commit 87a0bd9
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion scripts/ci/pre_commit/pre_commit_compile_www_assets.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/usr/bin/env python
#!/usr/bin/env python3
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
Expand Down
2 changes: 1 addition & 1 deletion scripts/ci/pre_commit/pre_commit_compile_www_assets_dev.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/usr/bin/env python
#!/usr/bin/env python3
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
Expand Down
2 changes: 1 addition & 1 deletion scripts/ci/pre_commit/pre_commit_www_lint.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/usr/bin/env python
#!/usr/bin/env python3
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
Expand Down

0 comments on commit 87a0bd9

Please sign in to comment.