gobuffalo / pop

A Tasty Treat For All Your Database Needs
MIT License
1.43k stars 241 forks source link

Fizz v4.5.10 type issue #138

Closed ni-kit closed 6 years ago

ni-kit commented 6 years ago

Hi everyone. I ran against an issue with fizz after upgrade to v4.5.10: Error: error processing migrations/booms.up.fizz: could not fizz the migration migrations/booms.up.fizz: parse error: function run error: function wants argument type fizz.Options but received type map[string]interface {} Had to add a constraint to resolve it.

u007 commented 6 years ago

i think u may need to go get -u github.com/gobuffalo/fizz and also go get -u to update pop

flexlee commented 6 years ago

I ran into the same issue. go get -u github.com/gobuffalo/fizz doesn't help.

mclark4386 commented 6 years ago

This is a known bug that is caused by the anko package changing. We are in the process of moving to plush so this doesn't happen again, but in the mean time if you are using dep you can lock anko to 0.0.4

stanislas-m commented 6 years ago

go get -u github.com/gobuffalo/fizz won't help, since the concerned branch is not merged yet. go get -u github.com/gobuffalo/pop should work, though.

flexlee commented 6 years ago

@mclark4386 isn't current master of pop already locked anko at v0.0.4? Sorry I'm a bit confused.

Gopkg.lock

[[projects]]
  name = "github.com/mattn/anko"
  packages = [
    "ast",
    "core",
    "parser",
    "vm"
  ]
  revision = "e9c5c0ca86cf129292c56ddbddc4fff027844d65"
  version = "v0.0.4"
markbates commented 6 years ago

Update: The fix for this is here https://github.com/gobuffalo/pop/pull/130

markbates commented 6 years ago

fixed with the release of pop v4.5.11 https://github.com/gobuffalo/pop/releases/tag/v4.5.11. Update your dependencies and you'll be good to go.

markbates commented 6 years ago

No need to pin anko any more, as it's been removed entirely! :)