search

regro / cf-scripts

Flagship repo for cf-regro-autotick-bot
Other
51 stars 74 forks source link

Swap out extensions for upstreams #766

Open CJ-Wright opened 4 years ago

CJ-Wright commented 4 years ago

Sometimes upstream changes the extension. It might be nice if the bot tried a few things (similar to the v stuff) to get a viable upstream.

beckermr commented 4 years ago

I think we should not automerge things like this

beckermr commented 4 years ago

Like the only thing that gets auto merged is a straight version bump

CJ-Wright commented 4 years ago

Sure, that may make this a bit more complex (how do we let the main migrator know that we swapped out the extension). Also we should have a list of extensions in order of preference (if the standard one doesn't work)

beckermr commented 4 years ago

This turns out to be very expensive so not done right now.

CJ-Wright commented 4 years ago

Yeah I think we need to put that hashing stuff onto threads, we're most likely waiting on server communication.

beckermr commented 4 years ago

Right. It was in a separate process in dask but I am having a ton of trouble with the pr.

beckermr commented 4 years ago

I think actually we should move the pr updating and the version updating to their own processes in separate repos. This will put us in the road to dynamodb while giving us more breathing room.