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

Writing integration tests for an ASP.NET Web API #2043

Closed

Conversation

megakevin
Copy link
Contributor

@megakevin megakevin commented May 8, 2024

Before submitting a pull request, make sure to complete the following checklist:

  • Include attribution for anything you don't own. This means images, but also code snippets or quotes you didn't write. Even if you do use your own image, please add an HTML comment with attribution so we know where it came from.
  • Make sure you have the latest upstream changes: git fetch upstream && git rebase upstream/main
  • Clean up your repo. Once you are done with all changes, including from reviewers, squash any commits you have into a single commit. Make sure you don't have any leftover commits from old blog posts. It saves time when the keepers of the blog don't need to rebase and squash your commits. If you need help with this, reach out to a keeper of the blog (who are listed in the End Point internal wiki under the page of that name).
  • Push all changes from your local repo to your fork on GitHub.
  • Create a pull request draft, then request review from your technical and style reviewers on your pull request (you should also message them on company chat, as not everyone checks their GitHub notifications regularly). If you have already had people review it, mention that in the pull request description. You can have your reviewers look at your post directly in GitHub, which will convert the Markdown nicely and even parse the frontmatter into a little table! See this example
  • Incorporate changes based on your reviewers' input.
    • Complete technical review
    • Complete style review
  • When you are done incorporating feedback and making any other changes, contact a keeper of the blog for final review and publishing. You can message on company chat or in a comment on your pull request (make sure to tag Seth @sethjensen1).

Once your blog post has been published to the world, proofread it again. You often see mistakes after publishing that you didn't notice before, and it's not too late to fix them!

@megakevin
Copy link
Contributor Author

Here's a new blog post @sethjensen1

Thanks for the review @juanpabloventoso

@sethjensen1
Copy link
Contributor

This is live. Thanks @megakevin!

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.

None yet

2 participants