Skip to content

Commit

Permalink
adding clarifying comments
Browse files Browse the repository at this point in the history
  • Loading branch information
s1113950 committed Dec 10, 2020
1 parent 2510f1a commit 4657979
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@ matrix:
# env: MODE=debops_common VER=2.10.0

# Sanity check against vanilla Ansible. One job suffices.
# https://github.com/dw/mitogen/pull/715#issuecomment-719266420 migrating to Azure for now
# https://github.com/dw/mitogen/pull/715#issuecomment-719266420 migrating to Azure for now due to Travis 50 min time limit cap
# azure lets us adjust the cap, and the current STRATEGY=linear tests take up to 1.5 hours to finish
# - python: "2.7"
# env: MODE=ansible VER=2.10.0 DISTROS=debian STRATEGY=linear

Expand Down
4 changes: 4 additions & 0 deletions mitogen/master.py
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,10 @@ def _py_filename(path):
if basepath in SPECIAL_FILE_PATHS:
return path, True

# return None, False means that the filename passed to _py_filename does not appear
# to be python, and code later will handle when this function returns None
# see https://github.com/dw/mitogen/pull/715#discussion_r532380528 for how this
# decision was made to handle non-python files in this manner
return None, False


Expand Down

0 comments on commit 4657979

Please sign in to comment.