Matthew Rothenberg

Results 42 issues of Matthew Rothenberg

There probably isn't too much to do here, but it requires a greater understanding on my part of what a typical Windows git CLI environment looks like, so help from...

help wanted
platform:windows

https://github.com/src-d/go-git This could make shelling out to git completely unnecessary, and could remove some of the complex code related to parsing git porcelain output. It could also be overkill beyond...

refactor

This will improve cross-platform support for future shells, and also hopefully allow the code to become more clean and understandable. As @stereokai showed in #14, currently Windows in PowerShell does...

enhancement
platform:windows

## Summary In version `0.6.2`, a `run interactively` step followed by a `the exit status should be` step would complete, allowing the results of the process to be examined (as...

documentation

This will improve the ergonomics of this library since we currently use an `interface{}` container type. However, that will require go1.18+, which will be a breaking change, thus the plan...

_(Documenting some of my conversations with @outkaj for broader discussion and async. Note this is a discussion related to the upcoming changes switching to sbt-git versioning, not what is currently...

Short on documentation and tests, and most likely contains grievous implementation errors in the corner cases since we aren't testing for them yet, but don't want to invest any time...

Reasons: - `sseserver` is very generic, and doesn't imply any of the nice functionality (namespacing support and fast performance) in this library. - `sseserver` can be confusing to spell, due...

help wanted

Some options: - https://github.com/azer/logger - https://github.com/rs/zerolog - https://github.com/sirupsen/logrus (thx @nerney) - https://github.com/uber-go/zap Logging will need to be documented in the README as well, since it's essential for someone who would...

This should help prevent most gotchas based on local developer environment, but isn't foolproof, see note below I found: > Note: Merely setting the target option does not guarantee that...