#1433 Allow to set both global settings in config file at the same time #1434

lapisangularis · 2020-11-20T17:03:00Z

This PR:

Fixes an issue, when both global- settings are set in config, which threw an exception before this fix
Covered by tests
Fixes Both global-ignore and global-ignoreSourceCodeByRegex cannot be set in infection.json file at the same time #1433

…e same time

maks-rafalko

Thank you @lapisangularis for fixing this issue 👍

…me (#1434)

infection#1433 Allow to set both global settings in config file at th…

fbbd436

…e same time

maks-rafalko approved these changes Nov 20, 2020

View reviewed changes

maks-rafalko merged commit 8c9a21e into infection:master Nov 20, 2020

maks-rafalko pushed a commit that referenced this pull request Nov 20, 2020

#1433 Allow to set both global settings in config file at the same ti…

6035c15

…me (#1434)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#1433 Allow to set both global settings in config file at the same time #1434

#1433 Allow to set both global settings in config file at the same time #1434

lapisangularis commented Nov 20, 2020 •

edited by maks-rafalko

maks-rafalko left a comment

#1433 Allow to set both global settings in config file at the same time #1434

#1433 Allow to set both global settings in config file at the same time #1434

Conversation

lapisangularis commented Nov 20, 2020 • edited by maks-rafalko

maks-rafalko left a comment

Choose a reason for hiding this comment

lapisangularis commented Nov 20, 2020 •

edited by maks-rafalko