Skip to content

Commit

Permalink
Make sure that only pypi dependencies are used (home-assistant#15490)
Browse files Browse the repository at this point in the history
  • Loading branch information
andrey-git authored and Jacob Mansfield committed Sep 4, 2018
1 parent c374cf0 commit ed63b38
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions script/gen_requirements_all.py
Original file line number Diff line number Diff line change
Expand Up @@ -182,6 +182,10 @@ def gather_modules():
for req in module.REQUIREMENTS:
if req in IGNORE_REQ:
continue
if '://' in req:
errors.append(
"{}[Only pypi dependencies are allowed: {}]".format(
package, req))
if req.partition('==')[1] == '' and req not in IGNORE_PIN:
errors.append(
"{}[Please pin requirement {}, see {}]".format(
Expand Down Expand Up @@ -257,7 +261,7 @@ def write_requirements_file(data):


def write_test_requirements_file(data):
"""Write the modules to the requirements_all.txt."""
"""Write the modules to the requirements_test_all.txt."""
with open('requirements_test_all.txt', 'w+', newline="\n") as req_file:
req_file.write(data)

Expand All @@ -275,7 +279,7 @@ def validate_requirements_file(data):


def validate_requirements_test_file(data):
"""Validate if requirements_all.txt is up to date."""
"""Validate if requirements_test_all.txt is up to date."""
with open('requirements_test_all.txt', 'r') as req_file:
return data == req_file.read()

Expand Down

0 comments on commit ed63b38

Please sign in to comment.