-
Notifications
You must be signed in to change notification settings - Fork 17
update Go #31
Comments
@davidspek Are you using chainer now? This repo is inactive and we would like to get more feedbacks from users and then determine what's the next step? This probably needs more refactoring later if we still want to maintain it |
@Jeffwan I am not, I just went through all the repo's and added the script and resulting dependabot configuration so they can stay up to date with their dependencies easier and remove security vulnerabilities. |
@davidspek Thanks. We may do some clean up on these repos. I think this repo is inactive and we can skip Dependabot for this one. I talked with few folks in Capital one in the past and they want to open source their internal dask operator at that time. It's been a while. If you org has plan to open source the one you attached, that would be great. |
@Jeffwan We aren’t using a Dask operator yet. But the one I linked to is on GitHub under the Apache 2 license :). |
Hi @kubeflow/wg-training-leads, While adding the script to generate the dependabot configuration I noticed that the go dependency files are not compatible. I'm guessing Go needs to be updated so that
go.mod
andgo.sum
are created rather thanGopkg.lock
andGopkg.toml
. Once that is done, runningmake build-dependabot
from the root of the repo will update the config and allow it to scan for go dependencies.#30
The text was updated successfully, but these errors were encountered: