Closed akorchyn closed 3 months ago
@race-of-sloths include
The update primarily involves refactoring and enhancing the Contract
and Server
modules. Key changes include replacing async-channel
with futures
, adding a comment_id
field to the PR
struct, and updating function signatures and return types across various files for better error handling and logging. These modifications aim to improve the system's robustness and maintainability.
Files | Change Summaries |
---|---|
Cargo.toml , server/Cargo.toml |
Replaced async-channel with futures library. |
contract/src/lib.rs |
Added comment_id argument to sloth_include function and updated PR::new call. |
contract/src/types.rs |
Added comment_id field to PR struct and updated constructor. |
contract/src/views.rs |
Updated PRInfo struct with comment_id and votes fields. |
server/src/api/github/mod.rs |
Changed reply function return type to Comment , added Debug trait to GithubClient , and more. |
server/src/api/near.rs |
Various alterations to exported or public entities. |
server/src/commands/common.rs |
Renamed ContextStruct to Context , updated return types of several functions. |
server/src/commands/exclude.rs |
Modified execute function in BotExcluded struct to include _check_info parameter. |
server/src/commands/merged.rs |
Updated PullRequestMerged struct's execute method to include info parameter. |
server/src/commands/mod.rs |
Simplified Context struct, introduced Event enum, and updated execute method. |
server/src/commands/pause.rs |
Refactored execute methods in BotPaused and BotUnpaused structs to include additional parameters. |
server/src/commands/score.rs |
Adjusted execute method in BotScored struct to include PRInfo parameter. |
server/src/commands/stale.rs |
Updated PullRequestStale struct's execute method to include pr_info parameter. |
server/src/commands/start.rs |
Refactored execute method in BotIncluded struct to include info parameter and return a Result . |
server/src/lib.rs |
Added pub mod messages; . |
server/src/main.rs |
Various alterations to exported or public entities. |
server/src/messages/mod.rs |
Added status_message method to PRInfo struct for generating status update messages. |
π In the land of code where futures gleam, PRs now with comments beam. Contexts renamed, methods refined, Robust and clear, all intertwined. With messages clear and status bright, Slothrace moves with agile might. π
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Hey there! π Your PR is now part of the Race of Sloths. Thanks for contributing! π
Current Status:
@race-of-sloths score [1,2,3,5,8,13]
.We'll keep this status updated as things progress. Thanks again for your awesome contribution! π
The PR has been merged, but it was not scored. The score process will be closed after 24 hours automatically.
@race-of-sloths INclUDE
The PR has been finalized. Thank you for your contribution! The scoring process is closed now.
β¦essage
Summary by CodeRabbit
New Features
comment_id
to various structs and methods to enhance PR tracking and commenting capabilities.status_message
method for detailed status updates on Pull Requests.Refactor
ContextStruct
toContext
and simplified its structure.Enhancements
instrument
attribute for better debugging and traceability.