Actions
action #98997
closed[tools] [smelt] [python3.6] Replace all older text formatting with f-strings
Start date:
2021-09-21
Due date:
2021-10-05
% Done:
0%
Estimated time:
Description
User story¶
The python linter now gives warnings against the use of the old .format()
method of formatting strings, proposing the use of f-strings, as they are faster and more readable. This has caused the smelt pipelines to fail. Instead of removing the warning, it's a better idea to use f-strings where applicable.
Related ticket: https://gitlab.suse.de/tools/smelt/-/issues/730
Acceptance criteria¶
- AC1: All uses of the old formatting method that make sense to be replaced by f-strings are replaced
- AC2: The SMELT pipelines pass without pylint failing due to
consider-f-strings
warning
Tasks¶
- see failing pipeline results for places to switch to f-strings (i.e. https://gitlab.suse.de/tools/smelt/-/jobs/595135)
- replace
.format()
occurrences with f-strings, consider the best way to accurately do so so that the behaviour doesn't change
Further details¶
Open mr at https://gitlab.suse.de/tools/smelt/-/merge_requests/782
Actions