Skip to content

TST: Use match for pytest.warns - #8433

Merged
pllim merged 1 commit into
astropy:masterfrom
pllim:fix-pytest-warn
Feb 19, 2019
Merged

pllim merged 1 commit into
astropy:masterfrom
pllim:fix-pytest-warn

Conversation

@pllim

@pllim pllim commented Feb 19, 2019

Copy link
Copy Markdown
Member

@codecov

codecov Bot commented Feb 19, 2019 •

Copy link
Copy Markdown

Codecov Report

Merging #8433 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8433      +/-   ##
==========================================
- Coverage   86.78%   86.78%   -0.01%     
==========================================
  Files         387      387              
  Lines       58123    58123              
  Branches     1060     1060              
==========================================
- Hits        50442    50440       -2     
- Misses       7066     7068       +2     
  Partials      615      615
Impacted Files Coverage Δ
astropy/utils/data.py 80.93% <0%> (-0.45%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c73640b...77da4f0. Read the comment docs.

@pllim

pllim commented Feb 19, 2019

Copy link
Copy Markdown
Member Author

Update: 32-bit passed

@pllim pllim added this to the v2.0.12 milestone Feb 19, 2019
@pllim
pllim merged commit 3ba6f0b into astropy:master Feb 19, 2019
@pllim
pllim deleted the fix-pytest-warn branch February 19, 2019 17:04
bsipocz pushed a commit that referenced this pull request Feb 20, 2019
TST: Use match for pytest.warns
bsipocz pushed a commit that referenced this pull request Feb 20, 2019
TST: Use match for pytest.warns
@pllim pllim modified the milestones: v2.0.12, 3.1.2 Feb 20, 2019
@pllim

pllim commented Feb 20, 2019

Copy link
Copy Markdown
Member Author

@bsipocz , I am moving this milestone to 3.x since you said that LTS uses an older pytest.

@bsipocz

bsipocz commented Feb 20, 2019

Copy link
Copy Markdown
Member

It requires 2.8+ and this works with that if my double checking was right (and I already backported it).

@bsipocz bsipocz modified the milestones: 3.1.2, v2.0.12 Feb 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants