Skip to content

baremetrics: Fix doc.md to match current bot documentation standards.#911

Open
hanyucrocks wants to merge 1 commit intozulip:mainfrom
hanyucrocks:fix-baremetrics-doc
Open

baremetrics: Fix doc.md to match current bot documentation standards.#911
hanyucrocks wants to merge 1 commit intozulip:mainfrom
hanyucrocks:fix-baremetrics-doc

Conversation

@hanyucrocks
Copy link
Copy Markdown

@hanyucrocks hanyucrocks commented Mar 29, 2026

Fixes: #910

The doc.md for the Baremetrics bot had several issues compared to
the style used by other bots (xkcd, GIPHY):

  • Remove the ## Developer Notes section, which is internal contributor
    guidance, not user-facing documentation.
  • Fix a duplicate-word typo (short short descriptions).
  • Reformat the Usage entry to use the inline example style.
  • Add the standard "Run this bot" link after the Setup steps.

Screenshot of the revised version:
image

Self-review checklist
  • Self-reviewed the changes for clarity and maintainability
    (variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

  • Explains differences from previous plans (e.g., issue description).
  • Highlights technical choices and bugs encountered.
  • Calls out remaining decisions and concerns.
  • Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

  • Each commit is a coherent idea.
  • Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

  • Visual appearance of the changes.
  • Responsiveness and internationalization.
  • Strings and tooltips.
  • End-to-end functionality of buttons, interactions and flows.
  • Corner cases, error conditions, and easily imagined bugs.

Remove the Developer Notes section, which is internal contributor
guidance rather than user-facing documentation. Fix a duplicate-word
typo in the Usage section, reformat the Usage entry to match the
inline example style used by other bots (xkcd, GIPHY), and add the
standard "Run this bot" link after the Setup steps.

Fixes zulip#910.
@hanyucrocks
Copy link
Copy Markdown
Author

I believe the CI failing is a flaky issue. Tagging @timabbott for your insights.

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.

baremetrics: doc.md doesn't follow current bot documentation standards

1 participant