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]

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 a5be002 into 3c65104 - 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 f040c25 into 3c65104 - 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 a1ebce2 into 3c65104 - 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 aa55d6e into 3c65104 - 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 86a5a07 into 3c65104 - 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 8d9bce0 into 3c65104 - 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?