Skip to content

Commit

Permalink
Add Reviewable as a code review tool (#975)
Browse files Browse the repository at this point in the history
* Add Reviewable as a code review tool

* Fix broken link in profiling.md
  • Loading branch information
fahhem authored Jul 12, 2023
1 parent 4ecd956 commit 71acc25
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 1 deletion.
12 changes: 12 additions & 0 deletions docs/code-reviews/tools.md
Original file line number Diff line number Diff line change
Expand Up @@ -60,3 +60,15 @@ Work with pull requests on Azure DevOps directly out of Visual Studio.
1. Double-click a pull request - the **Pull Request Details** open
1. Click on **Checkout** if you want to have the full change locally and have a more integrated experience
1. Go through the changes and make comments

## Web

### Reviewable: [Seamless multi-round GitHub reviews](https://home.reviewable.io/)

Supports multi-round GitHub code reviews, with keyboard shortcuts and more. VS Code extension is in-progress.

1. Visit the [Review Dashboard](https://reviewable.io/reviews) to see reviews awaiting your action, that have new comments for you, and more.
1. Select a Pull Request from that list.
1. Open any file in your browser, in Visual Studio Code, or any editor you've configured by clicking on your profile photo in the top-right
1. Select an editor under "External editor link template". VS Code is an option, but so is any editor that supports URI's.
1. Review the diff on an overall or per-file basis, leaving comments, code suggestions, and more
1 change: 0 additions & 1 deletion docs/observability/profiling.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,5 @@ Unfortunately, each profiler tool typically uses its own format for storing prof
- (Java, Go, Python, Ruby, eBPF) [Pyroscope](https://github.com/pyroscope-io/pyroscope) continuous profiling out of the box.
- (Java and Go) [Flame](https://github.com/VerizonMedia/kubectl-flame) - profiling containers in Kubernetes
- (Java, Python, Go) [Datadog Continuous profiler](https://www.datadoghq.com/product/code-profiling/)
- (Java, Python, Go, Node.js) [Instana](https://www.instana.com/blog/instana-announces-the-industrys-first-commercial-continuous-production-profiler/)
- (Go) [profefe](https://github.com/profefe/profefe), which builds `pprof` to provide continuous profiling
- (Java) [Eclipse Memory Analyzer](https://www.eclipse.org/mat/)

0 comments on commit 71acc25

Please sign in to comment.