Skip to content

Conversation

@wtho
Copy link
Collaborator

@wtho wtho commented Oct 2, 2019

  • update default jsdom enviroment to v15
  • enable AngularNoNgAttributesSnapshotSerializer by default
  • use better reference of astTransformers (always in preset instead of
    relative) to avoid user confusion in README

@wtho wtho added dependencies Pull requests that update a dependency file Enhancement ✨ labels Oct 2, 2019
@wtho wtho added this to the 8.0.0 milestone Oct 2, 2019
@ahnpnl
Copy link
Collaborator

ahnpnl commented Oct 3, 2019

I think we should add a comment somewhere to remind ourselves to remove jest env jsdom 15 once jest 25 is out.

@wtho
Copy link
Collaborator Author

wtho commented Oct 3, 2019

Will v15 be default from jest v25 on?

Can you add a reference to that? Couldn't find anything in the jest repo. Thanks!

@ahnpnl
Copy link
Collaborator

ahnpnl commented Oct 3, 2019

Here you go

@wtho wtho force-pushed the chore/jsdom15-enable-serializer branch from cefe9b9 to 3684c04 Compare October 15, 2019 12:06
@wtho
Copy link
Collaborator Author

wtho commented Oct 15, 2019

Adding a simple issue is a good way keep it in mind #325

Copy link
Collaborator

@ahnpnl ahnpnl left a comment

Choose a reason for hiding this comment

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

LGTM!

namely
* update default jsdom enviroment to v15
* enable AngularNoNgAttributesSnapshotSerializer by default
* use better reference of astTransformers (always in preset instead of
relative) to avoid user confusion in README
@wtho wtho force-pushed the chore/jsdom15-enable-serializer branch from 3684c04 to 9751527 Compare October 18, 2019 16:14
@wtho
Copy link
Collaborator Author

wtho commented Oct 18, 2019

force-push: rebase master branch to fix conflicts

@thymikee thymikee merged commit 8161ea7 into thymikee:master Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file Enhancement ✨

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants