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

test(jQuery): run tests with jQuery 2.1, 2.2 & 3.2 #15843

Closed
wants to merge 1 commit into from

Conversation

mgol
Copy link
Member

@mgol mgol commented Mar 22, 2017

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Test update.

What is the current behavior? (You can also link to an open issue here)
We test with jQuery 2.1, 2.2 & 3.1.

What is the new behavior (if this is a feature change)?
We test with jQuery 2.1, 2.2 & 3.2.

Does this PR introduce a breaking change?
No.

Please check if the PR fulfills these requirements

Other information:
Also, update AngularJS 1.5 mentions in the docs to 1.6.

Copy link
Member

@gkalpak gkalpak left a comment

Choose a reason for hiding this comment

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

LGTM except for the tutorial related bits.

"angular-route": "1.5.x",
"angular": "1.6.x",
"angular-mocks": "1.6.x",
"angular-route": "1.6.x",
Copy link
Member

Choose a reason for hiding this comment

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

Upgrading the tutorial form 1.5.x to 1.6.x requires more changes. There are probably more, but off the top of my head:

  • The code on https://github.com/angular/angular-phonecat needs to be updated to match the changes here.
  • Some hrefs (on some steps) need to take into account the new default hashPrefix (!).
  • The code that explicitly sets the hash prefix to ! needs to be removed (and maybe some text referring to it).

In any case, I would not make the tutorial changes as part of this PR.

Copy link
Member Author

@mgol mgol Mar 22, 2017

Choose a reason for hiding this comment

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

OK, I'll revert them here.

Do you think I can perhaps leave jQuery 3.1 -> 3.2 update? That shouldn't require any changes in user code.

Copy link
Member

Choose a reason for hiding this comment

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

In the tutorial, it was 2.2.x --> 3.2.x. Is 1.5.x compatible with jQuery 3.x?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, AngularJS 1.5 is tested with jQuery 2.1.4, 2.2.4 & 3.1.0.

Copy link
Member Author

Choose a reason for hiding this comment

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

But jQuery follows semver since 1.12/2.2 so there are no breaking changes in 3.2.1.

@mgol
Copy link
Member Author

mgol commented Mar 22, 2017

This PR is now blocked on #15851.

Also, update jQuery 2.2.x mentions in the tutorial to 3.2.x.
@mgol
Copy link
Member Author

mgol commented Mar 22, 2017

The PR is now unblocked, PTAL.

@mgol mgol closed this in dcdd5de Mar 27, 2017
@mgol mgol deleted the jquery-3.2 branch March 27, 2017 09:28
mgol added a commit that referenced this pull request Mar 27, 2017
Also, update jQuery 2.2.x mentions in the tutorial to 3.2.x.

Closes #15843
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.

3 participants