search

canjs / can-connect

Model layer utilities for every JavaScript framework! Assemble real-time, high performance, restful data connections.
https://canjs.com/doc/can-connect.html
MIT License
29 stars 16 forks source link

An in-range update of can-define is breaking the build 🚨 #462

Closed greenkeeper[bot] closed 5 years ago

greenkeeper[bot] commented 5 years ago

The dependency can-define was updated from 2.7.12 to 2.7.13.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

can-define is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Status Details - ❌ **continuous-integration/travis-ci/push:** The Travis CI build could not complete due to an error ([Details](https://travis-ci.org/canjs/can-connect/builds/525065415?utm_source=github_status&utm_medium=notification)).
Release Notes for Documentation updates

Improve documentation for propDefinitions.

Ref - #439

Commits

The new version differs by 3 commits.

  • c3ea493 2.7.13
  • 316b405 Merge pull request #439 from canjs/279-improve-prop-definition-prominance
  • bf7c82e Improve documentation for propDefinitions

See the full diff

FAQ and help There is a collection of [frequently asked questions](https://greenkeeper.io/faq.html). If those don’t help, you can always [ask the humans behind Greenkeeper](https://github.com/greenkeeperio/greenkeeper/issues/new).

Your Greenkeeper Bot :palm_tree:

greenkeeper[bot] commented 5 years ago

After pinning to 2.7.12 your tests are still failing. The reported issue might not affect your project. These imprecisions are caused by inconsistent test results.

greenkeeper[bot] commented 5 years ago

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 6 commits.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests are passing again with this update. Explicitly upgrade to this version 🚀

Commits

The new version differs by 12 commits.

  • 47137aa 2.7.15
  • b84dfc3 Merge pull request #454 from canjs/safe-mutation-check
  • ea13ae4 Upgrade tests to use QUnit 2
  • 230dc1b Merge branch 'master' into safe-mutation-check
  • 1914ebb Remove can-observation reference as the test that was using it was removed
  • 7f0d53d Clarify unsafe mutation warning in can-define regarding the value() behavior
  • 9cc835b Use full function naming for Travis job to work correctly
  • 2d89ba8 Clean up changed package refs.
  • ce0ebb7 Make the warning message for unsafe mutations more informative
  • 8530844 Dev-only tests for warnings
  • fe4726e placate linter
  • c020777 Use canSafelyMutate to generate a dev warning when setting during a getter

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests are passing again with this update. Explicitly upgrade to this version 🚀

Release Notes for Trying to set a getter that doesn't take `lastSet` throws an unhelpful error

When a getter had zero-arguments, it threw an unhelpful error: Error: can-reflect.setValue - Can not set value. With this fix, an improved warning has been added to give users a better idea of what they've done wrong.

#367

Commits

The new version differs by 15 commits.

  • db0b41c 2.7.16
  • da6574c Merge pull request #460 from canjs/357-getter-unhelpful-error
  • 9cf71af error message fixes
  • 47137aa 2.7.15
  • b84dfc3 Merge pull request #454 from canjs/safe-mutation-check
  • ea13ae4 Upgrade tests to use QUnit 2
  • 230dc1b Merge branch 'master' into safe-mutation-check
  • 1914ebb Remove can-observation reference as the test that was using it was removed
  • 7f0d53d Clarify unsafe mutation warning in can-define regarding the value() behavior
  • 9cc835b Use full function naming for Travis job to work correctly
  • 2d89ba8 Clean up changed package refs.
  • ce0ebb7 Make the warning message for unsafe mutations more informative
  • 8530844 Dev-only tests for warnings
  • fe4726e placate linter
  • c020777 Use canSafelyMutate to generate a dev warning when setting during a getter

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests are passing again with this update. Explicitly upgrade to this version 🚀

Commits

The new version differs by 19 commits.

  • 7fb1a36 2.7.17
  • 6163615 Merge pull request #462 from canjs/461-failing-test-ie11
  • ee76ef2 readded steal.js file
  • 106de3e regex testing fix
  • db0b41c 2.7.16
  • da6574c Merge pull request #460 from canjs/357-getter-unhelpful-error
  • 9cf71af error message fixes
  • 47137aa 2.7.15
  • b84dfc3 Merge pull request #454 from canjs/safe-mutation-check
  • ea13ae4 Upgrade tests to use QUnit 2
  • 230dc1b Merge branch 'master' into safe-mutation-check
  • 1914ebb Remove can-observation reference as the test that was using it was removed
  • 7f0d53d Clarify unsafe mutation warning in can-define regarding the value() behavior
  • 9cc835b Use full function naming for Travis job to work correctly
  • 2d89ba8 Clean up changed package refs.

There are 19 commits in total.

See the full diff

greenkeeper[bot] commented 5 years ago

Your tests are passing again with this update. Explicitly upgrade to this version 🚀

Release Notes for (Updated) Tests using canReflect.getName in assertions fail in IE11

Test failed due to canReflect.getName returning a certain value fail in IE11 because of how we name functions differently in IE11. With this fix, the tests use regular expressions to make the assertions less strict.

Updated More test with RegEx

Issue #461

Commits

The new version differs by 22 commits.

  • 8800be6 2.7.18
  • 1d3d2a3 Merge pull request #463 from canjs/461-failing-test-ie11-2
  • 8fcc406 ie11 more regex fixes
  • 7fb1a36 2.7.17
  • 6163615 Merge pull request #462 from canjs/461-failing-test-ie11
  • ee76ef2 readded steal.js file
  • 106de3e regex testing fix
  • db0b41c 2.7.16
  • da6574c Merge pull request #460 from canjs/357-getter-unhelpful-error
  • 9cf71af error message fixes
  • 47137aa 2.7.15
  • b84dfc3 Merge pull request #454 from canjs/safe-mutation-check
  • ea13ae4 Upgrade tests to use QUnit 2
  • 230dc1b Merge branch 'master' into safe-mutation-check
  • 1914ebb Remove can-observation reference as the test that was using it was removed

There are 22 commits in total.

See the full diff