#3260 fix pytest section #3267
#3260 fix pytest section #3267
Conversation
feuillemorte
added
type: bug
_pytest/config.py
Outdated
| sections = ['tool:pytest', 'pytest'] if is_cfg_file else ['pytest'] | ||
| for section in sections: | ||
| try: | ||
| inicfg = iniconfig[section] |
There was a problem hiding this comment.
we never deprecated [pytest] in custom configs, i believe we should either support both or target the features branch
There was a problem hiding this comment.
@RonnyPfannschmidt fixed
I thought that #3086 deprecates [pytest] section in custom configs too
There was a problem hiding this comment.
@feuillemorte it is deprecated, but we will remove it only in 4.0. We should start issuing warnings here as well if users are using [pytest] in setup.cfg files, see:
Lines 1266 to 1267 in 2008554
But we don't introduce new warnings in bug-fixes (we've broken test suites in the past because of that) so the introduction of this warning should be done in features, in a separate task.
There was a problem hiding this comment.
@nicoddemus I see, thanks!
Fixes #3260
Bugfix: When you pass option "-c custom_config.cfg" section [tool:pytest] is now parses. And print warning if there is [pytest] section.