Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Dependency updates 2018-11-12 #1332

Merged
merged 1 commit into from
Nov 13, 2018
Merged

Dependency updates 2018-11-12 #1332

merged 1 commit into from
Nov 13, 2018

Conversation

akostibas
Copy link
Contributor

@akostibas
Copy link
Contributor Author

It looks like a change in Pop has broken saving null relations. I opened an issue here: gobuffalo/pop#298

@akostibas
Copy link
Contributor Author

I'm leaving out the Go dependency updates for now so that we can get the JS security issue patched.

jim
jim approved these changes Nov 13, 2018
View reviewed changes
Copy link
Contributor

@jim jim left a comment

Choose a reason for hiding this comment

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

🚛

@akostibas akostibas merged commit 734294c into master Nov 13, 2018
@akostibas akostibas deleted the dependency-updates branch November 13, 2018 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jim jim jim approved these changes

@chrisgilmerproj chrisgilmerproj Awaiting requested review from chrisgilmerproj

@pjdufour-truss pjdufour-truss Awaiting requested review from pjdufour-truss

@dynamike dynamike Awaiting requested review from dynamike

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@akostibas @jim