[Notes] [Git][BuildStream/buildstream][jjardon/ci_fedora28] dev-requirements.txt: Remove obsolete comments

[Javier Jardón <gitlab mg gitlab com>]

Title: GitLab

Javier Jardón pushed to branch jjardon/ci_fedora28 at BuildStream / buildstream

Commits:

• e9962f54
  by Javier Jardón at 2018-08-12T03:21:44Z

  dev-requirements.txt: Remove obsolete comments
  
  This also fixes an error with pylint in setup.py:
  setup.py:226:19: R1718: Consider using a set comprehension (consider-using-set-comprehension)
  

2 changed files:

• dev-requirements.txt
• setup.py

Changes:

dev-requirements.txt

---

1		-# Pin coverage to 4.2 for now, we're experiencing
2		-# random crashes with 4.4.2
3	1	coverage == 4.4.0
4	2	pep8
5	3	pytest >= 3.1.0
...	...	@@ -8,5 +6,4 @@ pytest-datafiles
8	6	pytest-env
9	7	pytest-pep8
10	8	pytest-pylint
11		-# Provide option to run tests in parallel, less reliable
12	9	pytest-xdist

setup.py

---

...	...	@@ -223,8 +223,7 @@ def get_cmdclass():
223	223	#####################################################
224	224	setup_requires = set(['pytest-runner'])
225	225	with open('dev-requirements.txt') as dev_reqs:
226		- dev_requires = set([line for line in dev_reqs.read().split('\n')
227		- if not line.strip().startswith('#')])
	226	+ dev_requires = dev_reqs.read().splitlines()
228	227
229	228
230	229	#####################################################

—
View it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.