Skip to content
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.

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

Dual hook high performance #2

Merged
merged 18 commits into from
Nov 17, 2023
Merged

Dual hook high performance #2

merged 18 commits into from
Nov 17, 2023

Conversation

psionic-k
Copy link
Member

Let's check the CI while in the MELPA queue ;-)

Signed-off-by: Psionik K <[email protected]>
For user friendliness, I'm expsoing some alias to other weirdly named logging
tools that fit different concepts.

Guess if I had a manual, I could direct the user to these things. Oh well.

Signed-off-by: Psionik K <[email protected]>
This should make it easier on everyone.  I tried to get rid of every name that
included "command" and "log" more than once.

There were some other poor choices in there.

It's clear that window size is not even supported.

Signed-off-by: Psionik K <[email protected]>
This commit also defines many variables for markers to enable the more
sophisticated output.

See documentation for details

Signed-off-by: Psionik K <[email protected]>
Note, this commit had to combine updates to the formatting, which makes use of
the markers, and many new logic hunks which compare pre and post commands to
handle various edge cases.

Sorry it couldn't be split after I was done.

Signed-off-by: Psionik K <[email protected]>
Signed-off-by: Psionik K <[email protected]>
@psionic-k psionic-k force-pushed the dual-hook-high-performance branch 2 times, most recently from 9a4b2b6 to fc62efe Compare November 17, 2023 13:14
@psionic-k psionic-k merged commit 03df232 into master Nov 17, 2023
3 of 5 checks passed
@psionic-k psionic-k deleted the dual-hook-high-performance branch November 17, 2023 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant