Update decoration order and environment parsing fixes

[bannanc]

This PR is addressing issue #62 currently, but I'm going to take a look at LGTM and travis to see if I can address issues #61 and #64 also.

I need to see what is causing the tests to fail also.

[codecov[bot]]

Codecov Report

Merging #65 into master will decrease coverage by 0.4%. The diff coverage is 66.08%.

[bannanc]

This pull request fixes 7 alerts when merging a5be002ab30cda7b70c7ff4ac9d6e81c6c62b934 into 3c6510463a2e658ba1a80e3ddb40625b278d404c - view on LGTM.com

fixed alerts:

• 2 for Except block handles 'BaseException'
• 2 for Unused import
• 2 for Redundant assignment
• 1 for Testing equality to None

---

Comment posted by LGTM.com

[bannanc]

This pull request introduces 5 alerts and fixes 26 when merging f040c2534c548d082ddeeb665286603422c83a64 into 3c6510463a2e658ba1a80e3ddb40625b278d404c - view on LGTM.com

new alerts:

• 4 for __eq__ not overridden when adding attributes
• 1 for Comparison of identical values

fixed alerts:

• 7 for Testing equality to None
• 5 for Incomplete ordering
• 4 for Except block handles 'BaseException'
• 4 for Unused import
• 2 for Module is imported with 'import' and 'import from'
• 2 for Redundant assignment
• 1 for Unused local variable
• 1 for Inconsistent equality and hashing

---

Comment posted by LGTM.com

[bannanc]

When trying to fix lgtm I broke a few things and apparently added alerts so there's still a lot of work to do, it is not ready for review.

[bannanc]

This pull request introduces 5 alerts and fixes 26 when merging a1ebce29702e2fe99d1a09768ad05fcde040ae1d into 3c6510463a2e658ba1a80e3ddb40625b278d404c - view on LGTM.com

new alerts:

• 4 for __eq__ not overridden when adding attributes
• 1 for Comparison of identical values

fixed alerts:

• 7 for Testing equality to None
• 5 for Incomplete ordering
• 4 for Except block handles 'BaseException'
• 4 for Unused import
• 2 for Module is imported with 'import' and 'import from'
• 2 for Redundant assignment
• 1 for Unused local variable
• 1 for Inconsistent equality and hashing

---

Comment posted by LGTM.com

[bannanc]

This pull request introduces 5 alerts and fixes 26 when merging aa55d6e507684479cca96b1ed51f1482a8e64b82 into 3c6510463a2e658ba1a80e3ddb40625b278d404c - view on LGTM.com

new alerts:

• 4 for __eq__ not overridden when adding attributes
• 1 for Comparison of identical values

fixed alerts:

• 7 for Testing equality to None
• 5 for Incomplete ordering
• 4 for Except block handles 'BaseException'
• 4 for Unused import
• 2 for Module is imported with 'import' and 'import from'
• 2 for Redundant assignment
• 1 for Unused local variable
• 1 for Inconsistent equality and hashing

---

Comment posted by LGTM.com

[bannanc]

This pull request introduces 5 alerts and fixes 26 when merging 86a5a0710cfcc7b640a62620145bc057ff1915c6 into 3c6510463a2e658ba1a80e3ddb40625b278d404c - view on LGTM.com

new alerts:

• 4 for __eq__ not overridden when adding attributes
• 1 for Comparison of identical values

fixed alerts:

• 7 for Testing equality to None
• 5 for Incomplete ordering
• 4 for Except block handles 'BaseException'
• 4 for Unused import
• 2 for Module is imported with 'import' and 'import from'
• 2 for Redundant assignment
• 1 for Unused local variable
• 1 for Inconsistent equality and hashing

---

Comment posted by LGTM.com

[bannanc]

This pull request introduces 2 alerts and fixes 26 when merging 8d9bce05ea77bbc6f2131c66cb1c266acc40ff73 into 3c6510463a2e658ba1a80e3ddb40625b278d404c - view on LGTM.com

new alerts:

• 2 for __eq__ not overridden when adding attributes

fixed alerts:

• 7 for Testing equality to None
• 5 for Incomplete ordering
• 4 for Except block handles 'BaseException'
• 4 for Unused import
• 2 for Module is imported with 'import' and 'import from'
• 2 for Redundant assignment
• 1 for Unused local variable
• 1 for Inconsistent equality and hashing

---

Comment posted by LGTM.com

[bannanc]

@jmaat I'm going to merge this one too, I know it would be better for you to review first, but I want to get a version out before I leave this weekend and the order and parsing errors shouldn't be in that. Could you make sure this is on your list to check?