Skip to content

Bump raven-js dependency to test getsentry/raven-js#582 #4004

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 24, 2016

Conversation

benvinegar
Copy link
Contributor

@benvinegar benvinegar commented Aug 24, 2016

/cc @getsentry/team


This change is Reviewable

@@ -31,7 +31,7 @@
"platformicons": "0.1.0",
"po-catalog-loader": "^1.2.0",
"query-string": "2.4.2",
"raven-js": "3.5.0",
"raven-js": "git+https://github.com/getsentry/raven-js.git#22bebbdbb619183978761dbb68fcb59ae8b3b099",
Copy link
Contributor

Choose a reason for hiding this comment

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

I personally prefer the getsentry/raven-js#22bebbdbb619183978761dbb68fcb59ae8b3b099 syntax myself.

@mattrobenolt
Copy link
Contributor

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.


Comments from Reviewable

@benvinegar benvinegar merged commit 1d89dd8 into master Aug 24, 2016
@benvinegar benvinegar deleted the test-ravenjs branch October 19, 2016 03:17
@github-actions github-actions bot locked and limited conversation to collaborators Dec 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants