Unpin pytest

[alimcmaster1]

Follow up from #26619. Since pytest have just done a point release for 4.6.1 we should be good to remove the pin.

pytest-dev/pytest#5358

cc @simonjayhawkins

[codecov]

Codecov Report

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

@@ Coverage Diff @@ ## master #26625 +/- ## ========================================== - Coverage 91.87% 91.87% -0.01%  ========================================== Files 174 174 Lines 50692 50692 ========================================== - Hits 46575 46571 -4  - Misses 4117 4121 +4

Flag	Coverage Δ
#multiple	90.41% <ø> (ø)	⬆️
#single	41.78% <ø> (-0.09%)	⬇️

Impacted Files	Coverage Δ
pandas/io/gbq.py	78.94% <0%> (-10.53%)	⬇️
pandas/core/frame.py	97% <0%> (-0.12%)	⬇️

---

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 6354580...d2beb74. Read the comment docs.

[codecov]

Codecov Report

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

@@ Coverage Diff @@ ## master #26625 +/- ## ========================================== - Coverage 91.88% 91.87% -0.01%  ========================================== Files 174 174 Lines 50692 50692 ========================================== - Hits 46576 46572 -4  - Misses 4116 4120 +4

Flag	Coverage Δ
#multiple	90.41% <ø> (ø)	⬆️
#single	41.8% <ø> (-0.09%)	⬇️

Impacted Files	Coverage Δ
pandas/io/gbq.py	78.94% <0%> (-10.53%)	⬇️
pandas/core/frame.py	97% <0%> (-0.12%)	⬇️

---

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 6354580...08c8159. Read the comment docs.

[alimcmaster1]

Interestingly I think there still might be an issue with the pytest release based on:

 def manual_compare_stacked(df, df_stacked, lev0, lev1): > assert all(df.loc[row, col] == df_stacked.loc[(row, col[lev0]), col[lev1]] for row in df.index for col in df.columns) E NameError: name 'col' is not defined 

will take a further look tomorrow

[alimcmaster1]

bumped up to latest pytest release which should fix this - CI running..

[gfyoung]

@jreback : How does it work when it comes to developer dependencies? Should we be bumping version requirements this high to be the latest version?

[jreback]

doesn’t make sense to bump this - we try to move very slowly in things like thus

[alimcmaster1]

Given that pandas tests will fail running with pytest version 4.6.0 and 4.6.1 doesn’t it make sense to enforce this in the dependencies? ( yes we could do this by keeping the current pin in place ).

But keeping it pinned means nothing in CI will draw our attention to any future breaking pytest changes? Potentially making it harder to update further down the road. What do you think ?

[jreback]

these are 3.5 builds only - pinning is ok

to be honest could pin them to a much earlier version
say 4.02 would be good (or for that would be ok)

all other builds float to the latest