pub struct Implementation {
pub not_worth_doing: bool,
pub reason: String,
pub summary: String,
pub problem: String,
pub changes: Vec<String>,
pub testing: Vec<String>,
pub notes: Option<String>,
}Expand description
What the implementor did, and what the pull request body is built from.
Structured for the reason every other exchange here is structured: a model asked for a description writes a paragraph about having written one, while a model asked for a problem, a change list, and a way to check them answers each of those. The body’s substance comes from the fields being asked for separately, and its brevity from spar composing them rather than the model narrating.
Fields§
§not_worth_doing: boolThe issue should not be implemented. No commits, and reason says why.
reason: StringOnly when declining. Posted on the issue, so it is written for whoever opened it rather than for the harness.
summary: StringOne sentence saying what changed. Leads the body.
problem: StringWhat was actually wrong, as understood after reading the code. Not a restatement of the issue: the reviewer can follow the link.
changes: Vec<String>One line per change that alters behaviour.
testing: Vec<String>How a reviewer confirms the change works.
notes: Option<String>Anything the reviewer would otherwise have to ask about: a deliberate omission, a decision worth defending, a risk. Usually nothing.
Trait Implementations§
Source§impl Clone for Implementation
impl Clone for Implementation
Source§fn clone(&self) -> Implementation
fn clone(&self) -> Implementation
1.0.0 (const: unstable) · Source§fn clone_from(&mut self, source: &Self)
fn clone_from(&mut self, source: &Self)
source. Read more