Colin Dean

Results 300 comments of Colin Dean

I'm going to hold off on merging this for a bit. After putting this into production ourselves yesterday afternoon, we've seen elevated disk space usage. I want to be sure...

We've not yet solved the disk usage problem but we did improve upon this function by enabling more flexibility in the `model()` or `model.predict()` call. I'll add to this PR...

> Hey, is this pull ready for production use? In theory, yes. I've not merged it because I wanted some more feedback before doing so since my case might have...

I'll work on this one.

Yeah, but someone else is welcome to take it if I don't finish it first.

https://github.com/simonmichael/hledger/pull/2159 maybe fixes the test failure, but probably not. I suspect that the base bash version in macOS might not run the script. I don't think it's in-bounds to establish...

Blocked on 1.32.3 release which should fix the hledger-bar error.

I found a bug in 1.32.3, too, so we'll have to wait until the next patch release for https://github.com/simonmichael/hledger/pull/2173 to be merged.

I'd like to see this. I like `Brewfile.last_run.json` as the new file name. I like writing to that file if nothing exists or writing to `Brewfile.lock.json` if it already exists....

> > I don't like altering files for users unless the program completely "owns" that file > > `brew bundle` does completely own this file. > > > so suggesting...