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

Update @segment/loosely-validate-event to the latest version 🚀 #152

Closed

Conversation

greenkeeper[bot]
Copy link
Contributor

@greenkeeper greenkeeper bot commented Jan 24, 2018

Version 2.0.0 of @segment/loosely-validate-event was just published.

Dependency @segment/loosely-validate-event
Current Version 1.1.2
Type dependency

The version 2.0.0 is not covered by your current version range.

If you don’t accept this pull request, your project will work just like it did before. However, you might be missing out on a bunch of new features, fixes and/or performance improvements from the dependency update.

It might be worth looking into these changes and trying to get this project onto the latest version of @segment/loosely-validate-event.

If you have a solid test suite and good coverage, a passing build is a strong indicator that you can take advantage of these changes directly by merging the proposed change into your project. If the build fails or you don’t have such unconditional trust in your tests, this branch is a great starting point for you to work on the update.


FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.


Your Greenkeeper bot 🌴

@codecov-io
Copy link

codecov-io commented Jan 24, 2018

Codecov Report

Merging #152 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #152   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          97     97           
=====================================
  Hits           97     97

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 46b0fbb...5fd2c7d. Read the comment docs.

@f2prateek
Copy link
Contributor

On hold because doing so will throw an error in analytics-node for messages > 32kb (previously this would be accepted).

I think we have two options here when updating to 2.0.0:

  • Make a major release for analytics-node
  • Have a deprecation period where we catch the > 32kb error and log it. After 6 months, we flip the switch to throw the error like we do with other validation errors.

@greenkeeper greenkeeper bot closed this Mar 2, 2018
@greenkeeper greenkeeper bot deleted the greenkeeper/@segment/loosely-validate-event-2.0.0 branch March 2, 2018 22:20
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.

2 participants