Skip to content

docs: remove misleading link from "Don't over-optimize" section #720

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.

Sign up for GitHub

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

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

docs: remove misleading link from "Don't over-optimize" section #720

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 3 additions & 4 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -977,10 +977,9 @@ function combine(val1, val2) {
### Don't over-optimize

Modern browsers do a lot of optimization under-the-hood at runtime. A lot of
times, if you are optimizing then you are just wasting your time. [There are good
resources](https://github.com/petkaantonov/bluebird/wiki/Optimization-killers)
for seeing where optimization is lacking. Target those in the meantime, until
they are fixed if they can be.
times, if you are optimizing then you are just wasting your time. Use browser
developer tools and performance profiling to identify real bottlenecks before
optimizing. Target those in the meantime, until they are fixed if they can be.

**Bad:**

Expand Down