Skip to content

Conversation

@blech75
Copy link
Member

@blech75 blech75 commented Dec 1, 2023

Thinking about how to be more direct w/r/t our prompts. Comments/discussion encouraged.

@@ -1,4 +1,4 @@
#### What does this PR do?
#### What changes are we making? Why are we making them?

Choose a reason for hiding this comment

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

From the peanut gallery: "What changes are we making? What problem do these changes solve? Why solve the problem this way?"

Copy link
Member Author

@blech75 blech75 Dec 1, 2023

Choose a reason for hiding this comment

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

i would actually love to have a separate section for each of these questions/prompts because it provides a framework for replying.

how about something like this? (slightly reworded)

What problem does this PR solve?

...

What changes are we making?

...

Why solve the problem this way?

...

perhaps there's a kinder way to ask about the rationale than Why solve the problem this way? — it comes across very confrontational, or at least that's how i read it. 😆

Copy link
Member Author

@blech75 blech75 Dec 1, 2023

Choose a reason for hiding this comment

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

maybe: What changes are we making? →

How are we solving the problem? (Explain the changes.)

...

that gives us a natural what → how → why progression

Copy link
Member

Choose a reason for hiding this comment

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

I think "What changes are we making?" could be taken too literally.

"What technical or business problems does this PR solve?" might be all that's needed IMO. That too me is the crux of what this section is asking.

Copy link
Member Author

Choose a reason for hiding this comment

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

how about:

What technical/business problem(s) are we solving?

...

How are we solving it? (Explain the changes and the rationale)

...

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.

4 participants