search

CCI-MOC / hil

Hardware Isolation Layer, formerly Hardware as a Service
Apache License 2.0
24 stars 54 forks source link

Ci cleanup #927

Closed naved001 closed 6 years ago

naved001 commented 6 years ago

Some of the yellow text in the ci was bothering me for a while so decided to clean it up before it becomes failures in the future.

The commit messages have info in them.

One message that is still annoying is this:

=============================================================================================== warnings summary ===============================================================================================
None
  pytest-catchlog plugin has been merged into the core, please remove it from your requirements.

-- Docs: http://doc.pytest.org/en/latest/warnings.html

I removed pytest-catchlog from the requirements, but then I couldn't import pytest_catchlog. Their docs don't say anything about it being merged into the core, it still tells you to install it explicitly. Don't know what's up with that.

naved001 commented 6 years ago

Note: only the tool name has changed from pep8 to pycodestyle (they had a slightly heated debate).

The document name PEP-8 still exists, and idk if that will be renamed or not.

I also added one more commit to use caplog.records property.

@zenhack @mosayyebzadeh made the changes you guys requested.

zenhack commented 6 years ago

LGTM

naved001 commented 6 years ago

2 sign offs, merging!