Skip to content

Update README.md#540

Merged
skrawcz merged 1 commit intomainfrom
skrawcz-patch-2
May 27, 2025
Merged

Update README.md#540
skrawcz merged 1 commit intomainfrom
skrawcz-patch-2

Conversation

@skrawcz
Copy link
Contributor

@skrawcz skrawcz commented May 27, 2025

Moving Margaret to the right spot, my bad.

[Short description explaining the high-level reason for the pull request]

Changes

How I tested this

Notes

Checklist

  • PR has an informative and human-readable title (this will be pulled into the release notes)
  • Changes are limited to a single goal (no scope creep)
  • Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future TODOs are captured in comments
  • Project documentation has been updated if adding/changing functionality.

Important

Move "Margaret Lange" from "Bug hunters/special mentions" to "Code contributions" in README.md.

  • Documentation:
    • Move "Margaret Lange" from "Bug hunters/special mentions" to "Code contributions" in README.md.

This description was created by Ellipsis for 5d4c4f5. You can customize this summary. It will automatically update as commits are pushed.

Moving Margaret to the right spot, my bad.
@skrawcz skrawcz merged commit 1965f49 into main May 27, 2025
11 checks passed
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 5d4c4f5 in 30 seconds. Click for details.
  • Reviewed 17 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. README.md:231
  • Draft comment:
    Good move: Margaret Lange is now correctly listed under Code contributions.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
2. README.md:237
  • Draft comment:
    Removed duplicate Margaret Lange entry from Bug hunters section since she belongs in the Contributors section.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_BNqiqmzboko9CMjL

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@skrawcz skrawcz deleted the skrawcz-patch-2 branch May 27, 2025 04:05
@github-actions
Copy link

github-actions bot commented May 27, 2025

A preview of is uploaded and can be seen here:

https://burr.dagworks.io/pull/540

Changes may take a few minutes to propagate. Since this is a preview of production, content with draft: true will not be rendered. The source is here: https://github.com/DAGWorks-Inc/burr/tree/gh-pages/pull/540/

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.

1 participant