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
introduce future and depracted options, and log.future #838
introduce future and depracted options, and log.future #838
Conversation
@boegel is this what you wanted? (sans test ofcourse) |
@@ -36,13 +36,18 @@ | |||
import sys | |||
from copy import copy | |||
from vsc import fancylogger | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wai you drop empty line? we're purposely splitting easybuild
imports from the others
@stdweird: I had something slightly different in mind, but this works too (and is probably better). I would just go for |
@boegel addressed remarks |
) | ||
try: | ||
log.experimental('x') | ||
except: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
except easybuild.tools.build_log.EasyBuildError, err:
?
Jenkins: please test this |
…ild-framework into deprecated_future_option
use log.experimental for format v2 usage, and enable use of experimental is relevant unit tests
@boegel merged in |
going in, thanks @stdweird! |
introduce future and depracted options, and log.future
No description provided.