Skip to content

docs: Update instructions in README#43

Merged
cademirch merged 1 commit intosnakemake:mainfrom
jlumpe:instructions
Mar 9, 2026
Merged

docs: Update instructions in README#43
cademirch merged 1 commit intosnakemake:mainfrom
jlumpe:instructions

Conversation

@jlumpe
Copy link
Copy Markdown
Contributor

@jlumpe jlumpe commented Oct 5, 2025

Tried to write out a more complete set of instructions in the README. A lot is just the comments previously in the code example, moved to their own section and with Markdown formatting applied.

Summary by CodeRabbit

  • Documentation
    • Added step-by-step guide to implement and register a logger plugin.
    • Introduced a migration guide from legacy log-handler scripts, with examples.
    • Documented available log events and typical fields, including usage examples.
    • Simplified examples and normalized terminology for better readability.
    • Streamlined references to configuration options to reduce ambiguity.
    • Reorganized README with a dedicated Instructions section for quicker onboarding.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Oct 5, 2025

📝 Walkthrough

Walkthrough

README.md was updated to reorganize and expand logger plugin documentation, including simplified settings examples, streamlined handler docs, added implementation/registration instructions, a migration guide from log-handler scripts, and a new section detailing available log events and typical fields.

Changes

Cohort / File(s) Summary of changes
Docs: Logger plugin guide
README.md
Removed advanced LogHandlerSettings metadata keys from examples; simplified LogHandler-related comments; normalized property/method docstrings; added instructions for plugin registration and implementation; added migration guide from log-handler scripts with examples; added “Available Log Events” section with event-field guidance and usage examples.

Sequence Diagram(s)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • docs: update readme #29 — Also reorganizes and expands logger plugin README, including migration guide and LogHandler/Settings details.
  • feat!: API simplification #9 — Adjusts logger plugin surface, touching LogHandler/LogHandlerBase behavior and LogHandlerSettings structure.

Suggested reviewers

  • cademirch
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Title check ✅ Passed The title 'docs: Update instructions in README' accurately describes the main change, which is moving and expanding documentation with new instruction sections in the README.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cademirch cademirch changed the title Update instructions in README docs: Update instructions in README Mar 9, 2026
@cademirch cademirch merged commit ad41bb5 into snakemake:main Mar 9, 2026
4 of 6 checks passed
@jlumpe jlumpe deleted the instructions branch March 15, 2026 07:16
johanneskoester pushed a commit that referenced this pull request Mar 19, 2026
🤖 I have created a release *beep* *boop*
---


##
[2.0.1](v2.0.0...v2.0.1)
(2026-03-09)


### Documentation

* Update instructions in README
([#43](#43))
([ad41bb5](ad41bb5))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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