Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Bugfix - Typos #13507

Closed
wants to merge 2 commits into from
Closed

Bugfix - Typos #13507

wants to merge 2 commits into from

Conversation

jutaz
Copy link
Contributor

@jutaz jutaz commented Dec 11, 2015

This PR fixes few typo found around the place

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@jutaz
Copy link
Contributor Author

jutaz commented Dec 11, 2015

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@@ -47,7 +47,7 @@ install:
# Log HTTP requests
- npm config set loglevel http
- npm install -g npm@2.5
# Instal npm dependecies and ensure that npm cache is not stale
# Instal npm dependencies and ensure that npm cache is not stale
Copy link
Contributor

Choose a reason for hiding this comment

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

There' also instal -> install

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Haha, missed this one!

@jutaz jutaz force-pushed the bugfix/typo-fixes branch from e0bdaa5 to 99a154d Compare December 11, 2015 10:21
@jutaz
Copy link
Contributor Author

jutaz commented Dec 11, 2015

Amended to include Instal -> Install

@@ -261,7 +261,7 @@ function computeCssStyles($window, element, properties) {
}

// by setting this to null in the event that the delay is not set or is set directly as 0
// then we can still allow for zegative values to be used later on and not mistake this
// then we can still allow for negative values to be used later on and not mistake this
Copy link
Member

Choose a reason for hiding this comment

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

Maybe zegative means zero or negative 😛

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But from what I understand, zero values are being set to null, right? So in that case zero is not allowed 😉

Copy link
Member

Choose a reason for hiding this comment

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

Yes, but it could be a bug in the code and not a typo in the docs 😛
Just kidding...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Haha, that would be bad

@gkalpak
Copy link
Member

gkalpak commented Dec 11, 2015

Nice one !

I favour (pun intended) cancelled / behaviour over canceled / behavior, but that's unimportant :)

@gkalpak gkalpak added this to the 1.5.x - migration-facilitation milestone Dec 11, 2015
@jutaz
Copy link
Contributor Author

jutaz commented Dec 11, 2015

I favour (pun intended) cancelled / behaviour over canceled / behavior, but that's unimportant :)

Yeah, but given that this project is EN_us, it makes sense, as behaviour and cancelled are more from EN_gb locale 😉

@gkalpak gkalpak closed this in e57cf13 Dec 11, 2015
@gkalpak
Copy link
Member

gkalpak commented Dec 15, 2015

BTW this is not "backportable" to v1.4.x.
If you care to resolve the conflicts and submit a PR against the v1.4.x branch (since you seem to be really serious about typos), I'd be happy to merge 😃

Thx for all the typo crashing so far 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants