Skip to content

Rewrite in go #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
jtyers wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Rewrite in go #1

jtyers wants to merge 30 commits into from

Conversation

@jtyers
Copy link
Owner

@jtyers jtyers commented Jul 6, 2020

Wholesale re-write in Go.

Benefits:

  • install as a single binary, simples
  • can add it to $PATH rather than having to update your .bashrc/.zshrc
  • faster
  • probably more robust and maintainable
  • we can actually write tests

jtyers and others added 30 commits June 28, 2020 07:46
@jtyers
feat: initial code for a simple TUI ww
b3916d0
@jtyers
feat: refactored to use channels rather than direct writes
b0e0472
@jtyers
fix: various fixes; better layout, and interval support
7ea126f
@jtyers
fix: layout and clipping fix
663f9da
@jtyers
feat: use default term color rather than white for output
16664e1
@jtyers
feat: added -s / --shell for wrapping cmd in a shell (derived from
dbb2327 
$SHELL)
@jtyers
feat: allow triggers to feed back status updates before they fire
be36c34
@jtyers
feat: moved to a better CLI args parser
bd69cf3
@jtyers
fix: trigger test fixes
a633640
@jtyers
feat: set background color of top row rather than ust text
8a9127f
@jtyers
feat: added highlighting
23e766a
@jtyers
fix: changed -H and --highlight to --color, --colour, -c to match cur…
4ee7ad9 
…rent ww usage
@jtyers
feat: added parser for WW_DEFAULT_ARGS
cafa11a
@jtyers
feat: add WW_DEFAULT_ARGS support into ww
889663d
@jtyers
feat: move to flaggy for args parsing to get sne behaviour on -h and …
b1aeea5 
…unknown args
@jtyers
feat: moved from channels to callbacks in ww
0fcacf5
@jtyers
fix: remove config printinng
82cd224
@jtyers
feat: re-jigged scanning logic so we detect EOFs correctly, and wait …
cba35cf 
…for EOFs before Wait()
@jtyers
tidy: refactor status callback using a custom struct for readability,
f9e7776 
refactor inline funcs to struct funcs
@jtyers
fix: use channels to synchronise stdout/stderr scanners
d67cf58
@jtyers
feat: added fsnotify (file system watch) support, and refactored ever…
6e3a830 
…ything
@jtyers
feat: add support for watch exclusions
e400b43
@jtyers
fix: ignore .git by default for --watch
8c7a896
@jtyers
fix: do not re-use watchers, re-create every time;
839124e
@jtyers
fix: fix update of text view on command re-run
f246887
@jtyers
feat: refactor tview out of WW into abstract WWDisplay
85af840
@jtyers
feat: add new UiLiveDisplay which does not go full screen (on by defa…
fc7d6ab 
…ult,

  switch to tview display using -f/--fullscreen)
@jtyers
feat: make shell processing the default, and can be disabled with -S
105d334
@jtyers
feat: add color for uilive display
892dde2
@jtyers
tidy: go mod tidy
b5bab64
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jtyers