Skip to content

style: simplify string formatting for readability - #649

Open
hirehamir wants to merge 1 commit into
tamasfe:masterfrom
hirehamir:style/simplify-string-formatting-for-readability
Open

hirehamir wants to merge 1 commit into
tamasfe:masterfrom
hirehamir:style/simplify-string-formatting-for-readability

Conversation

@hirehamir

Copy link
Copy Markdown

No description provided.

@hirehamir hirehamir left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't result in any user-facing changes. The goal of this change is to improve code readability and maintainability.

@hirehamir

Copy link
Copy Markdown
Author

@thernstig what are your thoughts on this change?

@hirehamir

Copy link
Copy Markdown
Author

Is there anything I can do to help get this merged?

@thernstig

Copy link
Copy Markdown
Contributor

@hamirmahal I am not an owner or maintainer of this repo. Try pinging one of the owners.

@hirehamir

Copy link
Copy Markdown
Author

Ah, sorry for the noise.

@hirehamir

Copy link
Copy Markdown
Author

@tamasfe what are your thoughts on this change?

@hirehamir

Copy link
Copy Markdown
Author

@panekj are you open to this change?

@hirehamir

Copy link
Copy Markdown
Author

@panekj I wanted to follow up since it's been over a year.

Are there any changes you'd like to see here?

This branch has not been deployed

No deployments
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