Skip to content

Added "Help Me Decide", changed wording about DR#449

Open
Expertium wants to merge 5 commits intoankitects:mainfrom
Expertium:patch-18
Open

Added "Help Me Decide", changed wording about DR#449
Expertium wants to merge 5 commits intoankitects:mainfrom
Expertium:patch-18

Conversation

@Expertium
Copy link
Contributor

#383 is inactive, so I made this
@Danika-Dakika feedback is welcome

@Expertium Expertium changed the title Added "Help Me Decide", changed wording about DR, removed mentions of… Added "Help Me Decide", changed wording about DR Sep 16, 2025
@Expertium
Copy link
Contributor Author

Expertium commented Oct 5, 2025

@dae PR is ready

Copy link
Contributor

@GithubAnon0000 GithubAnon0000 left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

@GithubAnon0000 GithubAnon0000 left a comment

Choose a reason for hiding this comment

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

I didn't check for that before but due to your latest commit I noticed it. It would be better if the lines are broken after they hit the 80 character column limit. That way, e.g. small changes like your last commit will be easier to compare (and the text / code will be easier to read).

Would be great if you could fix the strings to be less than 80 characters per line.

@Expertium
Copy link
Contributor Author

Well, seems like this will take a while to get merged

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