Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clarify prompts #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

clarify prompts #1

wants to merge 1 commit into from

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.

3 participants