jmcvetta/neoism

Error building newest version

Closed this issue · 3 comments

troq commented

I get this error

../../jmcvetta/neoism/entity.go:61: not enough arguments in call to e.Db.Session.Delete

Looks to be because Delete requires url string, p *url.Values, result, errMsg interface{} now instead of url string, result, errMsg interface{}.

👍 thanks @troq

The immediate issues here was, just as @troq said, the change in number of arguments expected by Napping's Delete() method.

However in the process of fixing this I discovered another bug in Napping that had been introduced unnoticed in a recent PR merge. That bug prevented tests from passing even when the aforementioned API issue was corrected. Napping has been updated to correct this problem.

This bug should now be fixed in Neoism v1.1.1, the current master.

troq commented

Thanks @jmcvetta!