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

Blogposts #1314

Merged
merged 2 commits into from
Oct 28, 2023
Merged

Blogposts #1314

merged 2 commits into from
Oct 28, 2023

Conversation

MinThaMie
Copy link
Contributor

@MinThaMie MinThaMie commented Oct 26, 2023

Ember Times review template

  • Add your name to top and bottom
  • Put emoji in writeup title
  • Add a short blurb (could be the title) to the beginning
  • Link to external article/repo/etc in paragraph/body text, not just the writeup title (link in writeup title is optional now)
  • Add the contributor in the post in format "FirstName LastName (@githubusername)" linked to their GitHub account
  • Check that all links work

@MinThaMie MinThaMie requested a review from a team as a code owner October 26, 2023 14:31
@probot-autolabeler probot-autolabeler bot added the embertimes PR for an Ember Times issue label Oct 26, 2023
@jaredgalanis jaredgalanis merged commit b93b773 into ember-times-208 Oct 28, 2023
2 checks passed
@jaredgalanis jaredgalanis deleted the blogposts branch October 28, 2023 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
embertimes PR for an Ember Times issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants