From 4657979210adbf3987c98cf320eb12d27c4e5914 Mon Sep 17 00:00:00 2001 From: Steven Robertson Date: Wed, 9 Dec 2020 22:51:46 -0800 Subject: [PATCH] adding clarifying comments --- .travis.yml | 3 ++- mitogen/master.py | 4 ++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index c080cf61..aafb4413 100644 --- a/.travis.yml +++ b/.travis.yml @@ -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 diff --git a/mitogen/master.py b/mitogen/master.py index 95b9ae9b..e54795cb 100644 --- a/mitogen/master.py +++ b/mitogen/master.py @@ -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