standard-test-roles

#76 Stick to the pep8 style standard-inventory-local
Closed by astepano. Opened by sturivny.
Unknown source master

Download 76.patch

Remove unused import.
Groome code to pep8 style.

Updated standard-inventory-local.py file
Edited by sturivny

1 new commit added

  • Fix exception flow in standard-inventory-local

It is a bit not easy to review PR with two commits. Two commits change the same code twice.

Second commit:

54   if __name__ == '__main__':
55 -     sys.exit(main())
56 +     main()

First commit:

50 -     sys.exit(main(sys.argv))
51 +     sys.exit(main())

This PR has two logicaly independent commits. One of the PEP8, other commit fixes exceptions.

Such PR requires a more time to review.

1 new commit added

  • Stick to the pep8 style

It is a bit not easy to review PR with two commits. Two commits change the same code twice.
Second commit:
54 if name == 'main':
55 - sys.exit(main())
56 + main()

First commit:
50 - sys.exit(main(sys.argv))
51 + sys.exit(main())

This PR has two logicaly independent commits. One of the PEP8, other commit fixes exceptions.
Such PR requires a more time to review.

Updated, now there only Stick to the pep8 style standard-inventory-local commit

Pull-Request has been closed by astepano
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.