Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

/about: Add Octobot's description #225

Closed
wants to merge 1 commit into from
Closed

Conversation

mctaylors
Copy link
Member

In this PR, I've added a description of Octobot to /about. Actually, /about is meant to have a description in it.

@mctaylors mctaylors added the type: feature Features and feature requests. label Dec 17, 2023
@mctaylors mctaylors requested a review from a team as a code owner December 17, 2023 23:35
@pull-request-size pull-request-size bot added the size/XS PRs that change <20 lines. label Dec 17, 2023
@Octol1ttle Octol1ttle added the status: wontfix Issues that will not be resolved, for one reason or another. label Dec 18, 2023
@Octol1ttle
Copy link
Member

You haven't provided a reason for this change, this change is redundant, the description may become out of sync with the About Me field. This PR suffers from the same issues as #224

@Octol1ttle Octol1ttle closed this Dec 18, 2023
@mctaylors
Copy link
Member Author

the description may become out of sync with the About Me field.

Just so you know: this description should synchronize manually with README.md, not with the "About me" section

@Octol1ttle
Copy link
Member

That makes the problem even worse. Having to repeat the same three times makes it easier to miss something being out of sync

@mctaylors
Copy link
Member Author

That makes the problem even worse. Having to repeat the same three times makes it easier to miss something being out of sync

Then I should display Discord bot's name (#224) and get description from "About Me" section.

@mctaylors mctaylors deleted the add-about-description branch December 23, 2023 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS PRs that change <20 lines. status: wontfix Issues that will not be resolved, for one reason or another. type: feature Features and feature requests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants