update dependencies #78

Closed
noerw wants to merge 2 commits from noerw:update-deps into main
Member

Update all dependencies (as I didnt find a way to update specific vendored modules). For example, this pulls the last 12 months of changes of go-sdk as will be needed for #54 ;)
Manual testing (in lack of a proper test suite) of all commands didn't reveal any obious problems.

GO111MODULE=on go get -u
make vendor
git add .
git commit

We could add a make task for this..

Update all dependencies (as I didnt find a way to update specific vendored modules). For example, this pulls the last 12 months of changes of `go-sdk` as will be needed for #54 ;) Manual testing (in lack of a proper test suite) of all commands didn't reveal any obious problems. ``` GO111MODULE=on go get -u make vendor git add . git commit ``` We could add a `make` task for this..
Author
Member

Can't see in the build log why the build failed..?

Can't see in the build log why the build failed..?

The error is: Please run 'make vendor' and commit the result:

The error is: `Please run 'make vendor' and commit the result:`
Author
Member

That's what I did.. (see above) ? Rerunning make vendor does nothing

That's what I did.. (see above) ? Rerunning `make vendor` does nothing

Which version of golang are you running? Perhaps if you are running go1.12 something has changed with how go1.13 generates the files?

Which version of golang are you running? Perhaps if you are running go1.12 something has changed with how go1.13 generates the files?
Author
Member

On the machine where I commited, I run go 1.12.13.
Just reran the above commands on another machine with go 1.13.5, and the results are indeed different.
Lets try

On the machine where I commited, I run go 1.12.13. Just reran the above commands on another machine with go 1.13.5, and the results are indeed different. Lets try
Author
Member

Can we rerun the tests? Seems to be a network timeout issue during build

Can we rerun the tests? Seems to be a network timeout issue during build
Owner

@noerw done.

@noerw done.
6543 approved these changes 2020-01-01 15:39:44 +00:00
Dismissed
6543 added a new dependency 2020-01-01 15:41:03 +00:00
6543 added this to the v0.2.0 milestone 2020-01-01 16:27:04 +00:00
Owner

@noerw fi you like to use 488204ae31 you shuld update dependencys again!

@noerw fi you like to use https://gitea.com/gitea/go-sdk/commit/488204ae3128ad6ed92629b593fce0b7ae53ba96 you shuld update dependencys again!
6543 requested changes 2020-01-01 16:31:47 +00:00
Dismissed
6543 left a comment
Owner

outdated!
gitea-sdk is now on 488204ae31

outdated! gitea-sdk is now on 488204ae31
Owner

In generall updatable:

  • code.gitea.io/sdk/gitea v0.0.0-20191013013401-e41e9ea72caa -> v0.0.0-20200103062250-c7686bd633c6
    -> #81
  • github.com/mattn/go-runewidth v0.0.4 -> v0.0.7
  • github.com/olekukonko/tablewriter v0.0.1 -> v0.0.4
  • github.com/sergi/go-diff v1.0.0 -> v1.1.0
  • github.com/stretchr/testify v1.3.0 -> v1.4.0 -> #83
  • github.com/urfave/cli v1.20.0 -> v1.22.2 -> #84
  • golang.org/x/crypto v0.0.0-20190701094942-4def268fd1a4 -> v0.0.0-20191227163750-53104e6ec876
  • golang.org/x/net v0.0.0-20190724013045-ca1201d0de80 -> v0.0.0-20191209160850-c0dbc17a3553
  • golang.org/x/sys v0.0.0-20190726091711-fc99dfbffb4e -> v0.0.0-20200103143344-a1369afcdac7
In generall updatable: * code.gitea.io/sdk/gitea v0.0.0-20191013013401-e41e9ea72caa -> v0.0.0-20200103062250-c7686bd633c6 -> #81 * github.com/mattn/go-runewidth v0.0.4 -> v0.0.7 * github.com/olekukonko/tablewriter v0.0.1 -> v0.0.4 * github.com/sergi/go-diff v1.0.0 -> v1.1.0 * github.com/stretchr/testify v1.3.0 -> v1.4.0 -> #83 * github.com/urfave/cli v1.20.0 -> v1.22.2 -> #84 * golang.org/x/crypto v0.0.0-20190701094942-4def268fd1a4 -> v0.0.0-20191227163750-53104e6ec876 * golang.org/x/net v0.0.0-20190724013045-ca1201d0de80 -> v0.0.0-20191209160850-c0dbc17a3553 * golang.org/x/sys v0.0.0-20190726091711-fc99dfbffb4e -> v0.0.0-20200103143344-a1369afcdac7
6543 removed this from the v0.2.0 milestone 2020-01-03 17:54:56 +00:00
6543 removed a dependency 2020-01-04 01:48:08 +00:00
6543 closed this pull request 2020-01-04 02:08:37 +00:00
Owner

@noerw if you need a specific dependency updated can you create a specific issue for that?

@noerw if you need a specific dependency updated can you create a specific issue for that?
Some checks are pending
continuous-integration/drone/pr Build is passing
check-and-test / check-and-test (pull_request)
Required

Pull request closed

Sign in to join this conversation.
No description provided.