Skip to content

Conversation

lunny
Copy link
Member

@lunny lunny commented Sep 14, 2025

No description provided.

@lunny lunny added the type/refactoring Existing code has been cleaned up. There should be no new functionality. label Sep 14, 2025
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 14, 2025
min_size_error = " must contain at least %s characters."
max_size_error = " must contain at most %s characters."
email_error = " is not a valid email address."
url_error = "\"%s\" is not a valid URL."
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrong.

Please fully understand the problem first.

Image

Copy link
Member Author

Choose a reason for hiding this comment

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

It was converted according to the first comment here.
image

Copy link
Contributor

Choose a reason for hiding this comment

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

No, it's wrong. INI package can't parse it. See the test cases

Copy link
Contributor

Choose a reason for hiding this comment

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

The INI package is totally a shit.

@lunny lunny marked this pull request as draft September 14, 2025 05:29
@silverwind
Copy link
Member

silverwind commented Sep 14, 2025

Maybe we can add a linter tool for this. Or switch to a sane format like json or yaml.

@wxiaoguang
Copy link
Contributor

It needs to rewrite the INI package.

@lunny
Copy link
Member Author

lunny commented Sep 14, 2025

replaced by #35489

@lunny lunny closed this Sep 14, 2025
@lunny lunny deleted the lunny/clean_en_us branch September 14, 2025 23:43
@lunny
Copy link
Member Author

lunny commented Sep 14, 2025

I sent #35489 to convert to JSON format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/translation type/refactoring Existing code has been cleaned up. There should be no new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants