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.

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

updated module dependencies #666

Closed
wants to merge 1 commit into from
Closed

Conversation

sio4
Copy link
Member

@sio4 sio4 commented Nov 20, 2021

As a part of gobuffalo/buffalo#2152, updated module dependencies.

Not sure if the development branch is ready to release, so I just PR it against master now. Let me know if I need to open a PR against development.

@sio4 sio4 requested a review from a team as a code owner November 20, 2021 02:37
Copy link
Member

@fasmat fasmat left a comment

Choose a reason for hiding this comment

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

See my comments.


replace github.com/mattn/go-sqlite3 => github.com/mattn/go-sqlite3 v1.14.7
replace github.com/mattn/go-sqlite3 => github.com/mattn/go-sqlite3 v1.14.9
Copy link
Member

Choose a reason for hiding this comment

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

@paganotoni This replace statement causes quite a bit of trouble and should be removed. This requires a temporary downgrade to fizz v1.10.0.

fizz can then be updated to use the newer version of pop without the replace statement, fixing the dependency graph of fizz.

Lastly this the new fizz version can be used here without the replace statement

I mention this issue in my PR: #667

Copy link
Member Author

Choose a reason for hiding this comment

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

Hi @fasmat,

I didn't know you are preparing PR for pop when I requested this PR. Please ignore this PR and please go ahead with your #667! I will close this PR for now!

@sio4 sio4 closed this Nov 21, 2021
@sio4 sio4 deleted the update-modules branch May 14, 2022 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants