Skip to content

Remove r-prefix of documentation string - #383

Closed
Marenz wants to merge 1 commit into
frequenz-floss:v0.x.xfrom
Marenz:no-r-prefix-kthxbye
Closed

Remove r-prefix of documentation string#383
Marenz wants to merge 1 commit into
frequenz-floss:v0.x.xfrom
Marenz:no-r-prefix-kthxbye

Conversation

@Marenz

@Marenz Marenz commented May 9, 2023

Copy link
Copy Markdown
Contributor

???

Signed-off-by: Mathias L. Baumann mathias.baumann@frequenz.com

@Marenz
Marenz requested a review from a team as a code owner May 9, 2023 08:55
@Marenz
Marenz requested a review from christianparpart May 9, 2023 08:55
@github-actions github-actions Bot added the part:power-management Affects the management of battery power and distribution label May 9, 2023
???

Signed-off-by: Mathias L. Baumann <mathias.baumann@frequenz.com>
@Marenz
Marenz force-pushed the no-r-prefix-kthxbye branch from d745a14 to a18d86f Compare May 9, 2023 08:58
@Marenz Marenz added the part:docs Affects the documentation label May 9, 2023
@Marenz Marenz self-assigned this May 9, 2023
@Marenz Marenz added this to the v0.21.0 milestone May 9, 2023
@leandro-lucarella-frequenz

Copy link
Copy Markdown
Contributor

This is just a raw string I think we can leave it as it is, as it's a very long a complex docstring.

https://docs.python.org/3/reference/lexical_analysis.html

Both string and bytes literals may optionally be prefixed with a letter 'r' or 'R'; such strings are called raw strings and treat backslashes as literal characters. As a result, in string literals, '\U' and '\u' escapes in raw strings are not treated specially. Given that Python 2.x’s raw unicode literals behave differently than Python 3.x’s the 'ur' syntax is not supported.

@Marenz Marenz closed this May 9, 2023
@Marenz
Marenz deleted the no-r-prefix-kthxbye branch May 9, 2023 13:15
@leandro-lucarella-frequenz leandro-lucarella-frequenz added the resolution:invalid This doesn't seem right label May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:docs Affects the documentation part:power-management Affects the management of battery power and distribution resolution:invalid This doesn't seem right

Projects

Development

Successfully merging this pull request may close these issues.

2 participants