Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ENH Set logger to "debug" so deprecations show #52

Merged

Conversation

emteknetnz
Copy link
Member

@emteknetnz emteknetnz commented Nov 2, 2022

Issue silverstripe/silverstripe-framework#10542
Other issue #53

I don't think there's any real downside since the silverstripe.log file never seems to show in the artifacts anyway even on red builds. There's more disk space used for artifacts, though I'd imagine that we're still far below whatever the quota is.

@emteknetnz
Copy link
Member Author

emteknetnz commented Nov 2, 2022

Actually, on second thought I don't want this - it's easy enough to turn this on custom branches on silverstripe/installer via yml - we already get all the error output we need in consoles, and this logging is only relevant for deprecation work, which will be done shortly anyway.

@emteknetnz emteknetnz closed this Nov 2, 2022
@emteknetnz
Copy link
Member Author

Reopening - we need for behat

@emteknetnz emteknetnz reopened this Nov 7, 2022
@emteknetnz emteknetnz changed the title ENH Set logger to "info" so deprecations show ENH Set logger to "debug so deprecations show Nov 8, 2022
@emteknetnz emteknetnz changed the title ENH Set logger to "debug so deprecations show ENH Set logger to "debug" so deprecations show Nov 8, 2022
@sabina-talipova sabina-talipova merged commit c2aca52 into silverstripe:1.2 Nov 15, 2022
@sabina-talipova sabina-talipova deleted the pulls/1.2/log-info branch November 15, 2022 01:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants