Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

Update qunitjs to version 2.0.1 🚀 #130

Closed
wants to merge 1 commit into from

Conversation

greenkeeperio-bot
Copy link
Contributor

Hello lovely humans,

qunitjs just published its new version 2.0.1.

State Update 🚀
Dependency qunitjs
New version 2.0.1
Type dependency

This version is not covered by your current version range.

Without accepting this pull request your project will work just like it did before. There might be a bunch of new features, fixes and perf improvements that the maintainers worked on for you though.

I recommend you look into these changes and try to get onto the latest version of qunitjs.
Given that you have a decent test suite, a passing build is a strong indicator that you can take advantage of these changes by merging the proposed change into your project. Otherwise this branch is a great starting point for you to work on the update.

Do you have any ideas how I could improve these pull requests? Did I report anything you think isn’t right?
Are you unsure about how things are supposed to work?

There is a collection of frequently asked questions and while I’m just a bot, there is a group of people who are happy to teach me new things. Let them know.

Good luck with your project ✨

You rock!

🌴


The new version differs by 100 commits (ahead by 100, behind by 1).

  • be660eb 2.0.1
  • 66faac3 Build: Prepare 2.0.1 release, including authors and history update
  • b7a00ac Core: Add extra guard against multiple begin()s
  • 933c721 Test: Use a single property for tracking all async deferred calls
  • 33e242b Core: Use callback-based pause/resume for better multi-pause isolation
  • 6e31bfc Test: Remove old async cruft
  • 434e8c3 Core: Better isolate async abstractions
  • 1743a14 Core: Revert source changes
  • 7c68498 Core: Improve internal async handling by removing resumed
  • c946161 HTML Reporter: Fix apply/reset button visibilty
  • 3cb21ef Core: Ensure runtime for skipped tests is 0
  • ba5f3e8 Dump: Add parser for Symbol
  • 680c2ca Build: Updating the master version to 2.0.1-pre.
  • b2817e2 Build: Prepare 2.0.0 release, including authors and history update
  • b2338e9 Tests: Fix script src for stack-errors test

There are 100 commits in total. See the full diff.


This pull request was created by greenkeeper.io.

Tired of seeing this sponsor message? ⚡ greenkeeper upgrade

@stefanpenner
Copy link
Contributor

@trentmwillis nice! Any reasons you can think of that would prevent us from merging this here?

@trentmwillis
Copy link
Member

Nope. This version was primarily released for a bug fix, so I'd encourage updating

@stefanpenner
Copy link
Contributor

@trentmwillis this upgrade is from "^1.20.0", -> "^2.0.1" I suspect that jump may be more then a bugfix? But if you think its good, lets merge it.

@trentmwillis
Copy link
Member

Ah didn't look at the actual diff. Assumed this had been updated to 2.0 already. But, now that I think about it, that wouldn't make sense because Ember breaks on QUnit 2.0 right now: emberjs/ember.js#13696

So hold off on merging. Whenever I finally have time to get Ember core compatible we'll merge.

@stefanpenner
Copy link
Contributor

Ah didn't look at the actual diff

I assumed, good thing I checked.

So hold off on merging. Whenever I finally have time to get Ember core compatible we'll merge.

Keep us posted.

@trentmwillis
Copy link
Member

Closing this in favor of #134.

@Turbo87 Turbo87 deleted the greenkeeper-qunitjs-2.0.1 branch August 14, 2016 08:47
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants