[iabhi4]

test only change - This PR removes the use of match="" in test_setitem_invalid within base/setitem.py.

Using an empty string as a match pattern triggers a PytestWarning in newer versions of pytest, so the match argument has been removed since the message was not being validated

• closes BUG: regex match in compliance tests no longer match pytest expected inputs #61557
• Ran pre-commit check

[rhshadrach]

lgtm

[datapythonista]

Personally I'd just remove the match parameter (unless we have a check to make sure it exists).

But either way this looks good, thanks for taking care of it @iabhi4.

[chalmerlowe]

Personally I'd just remove the match parameter (unless we have a check to make sure it exists).

But either way this looks good, thanks for taking care of it @iabhi4.

I tend to agree with @datapythonista, unless there is a specific check that we are looking for and in this case it does not appear to be, I would leave out the match parameter.

with pytest.raises((ValueError, TypeError)):

[datapythonista]

@mroeschke I think the changes you requested have been implemented, this should be ready to merge

[mroeschke]

Thanks @iabhi4