Skip to content

Namespaces #268

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Sep 19, 2018
Merged

Namespaces #268

merged 7 commits into from
Sep 19, 2018

Conversation

tiny-dancer
Copy link
Contributor

#265

@stenio123 was taking too long :)

version Outdated
@@ -1 +1 @@
0.6.4
0.6.5
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I missed that this change was made on this initial PR's commit. Would you mind getting it reverted @tiny-dancer? Pending that I see no reason we can't get this merged in. (FWIW, I am intending for a few other changes to go out in the 0.6.5 release that is tentatively slated for Oct. 1st.)

@tiny-dancer
Copy link
Contributor Author

@jeffwecan updated

@codecov-io
Copy link

Codecov Report

Merging #268 into master will decrease coverage by 0.05%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##           master    #268      +/-   ##
=========================================
- Coverage   89.96%   89.9%   -0.06%     
=========================================
  Files          16      16              
  Lines        1166    1169       +3     
=========================================
+ Hits         1049    1051       +2     
- Misses        117     118       +1
Impacted Files Coverage Δ
hvac/v1/__init__.py 85.55% <ø> (ø) ⬆️
hvac/adapters.py 95.52% <66.66%> (-1.36%) ⬇️

@tiny-dancer
Copy link
Contributor Author

@jeffwecan updated

@codecov-io
Copy link

Codecov Report

Merging #268 into master will decrease coverage by 0.05%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##           master    #268      +/-   ##
=========================================
- Coverage   89.96%   89.9%   -0.06%     
=========================================
  Files          16      16              
  Lines        1166    1169       +3     
=========================================
+ Hits         1049    1051       +2     
- Misses        117     118       +1
Impacted Files Coverage Δ
hvac/v1/__init__.py 85.55% <ø> (ø) ⬆️
hvac/adapters.py 95.52% <66.66%> (-1.36%) ⬇️

2 similar comments
@codecov-io
Copy link

Codecov Report

Merging #268 into master will decrease coverage by 0.05%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##           master    #268      +/-   ##
=========================================
- Coverage   89.96%   89.9%   -0.06%     
=========================================
  Files          16      16              
  Lines        1166    1169       +3     
=========================================
+ Hits         1049    1051       +2     
- Misses        117     118       +1
Impacted Files Coverage Δ
hvac/v1/__init__.py 85.55% <ø> (ø) ⬆️
hvac/adapters.py 95.52% <66.66%> (-1.36%) ⬇️

@codecov-io
Copy link

Codecov Report

Merging #268 into master will decrease coverage by 0.05%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##           master    #268      +/-   ##
=========================================
- Coverage   89.96%   89.9%   -0.06%     
=========================================
  Files          16      16              
  Lines        1166    1169       +3     
=========================================
+ Hits         1049    1051       +2     
- Misses        117     118       +1
Impacted Files Coverage Δ
hvac/v1/__init__.py 85.55% <ø> (ø) ⬆️
hvac/adapters.py 95.52% <66.66%> (-1.36%) ⬇️

Copy link
Member

@jeffwecan jeffwecan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect 👌

Thanks for getting this finalized!

@jeffwecan jeffwecan merged commit 1c2deb5 into hvac:master Sep 19, 2018
@jeffwecan jeffwecan modified the milestones: 0.7.0, 0.6.5 Sep 19, 2018
@tiny-dancer tiny-dancer deleted the namespaces branch September 19, 2018 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants