Skip to content

Docs: github templates#640

Open
adekunleoduye wants to merge 4 commits intosass:mainfrom
adekunleoduye:templates
Open

Docs: github templates#640
adekunleoduye wants to merge 4 commits intosass:mainfrom
adekunleoduye:templates

Conversation

@adekunleoduye
Copy link
Copy Markdown
Member

Summary

Created PR and Issue templates for PR.

Issue

#315

@nex3 nex3 self-requested a review June 8, 2022 20:04


### Issue
#
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this # intentional?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That was a placeholder to reference an issue or PR (i.e #640).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure that's clear from the template... maybe replace it with sass/sass-site#...?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Super late on this but made a change. Let me know what your thoughts on it.

Added placeholder to issue section.
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.

2 participants