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

worklog.py: Add a -s/--sort option that sorts output #1724

Merged
merged 1 commit into from
Feb 25, 2024

Conversation

ermo
Copy link
Contributor

@ermo ermo commented Feb 24, 2024

Summary

Without specifying the -s/--sort option, worklog.py will output the latest build or commit last.

This enables the ability to trivially keep an eye on large PR (re)builds with something like:

while true; do
    date
    ./worklog.py --format md builds 2024-02-24 now |tail -n5
    echo "----"
    sleep 30
done

Test Plan

Test the output of worklog.py with and without -s/--sort options.

Checklist

  • Package was built and tested against unstable

Without specifying the -s/--sort option, worklog.py will output the
latest build or commit last.

This enables the ability to trivially keep an eye on large PR (re)builds
with something like:

    while true; do
        date
        ./worklog.py --format md builds 2024-02-24 now |tail -n5
        echo "----"
        sleep 30
    done

Signed-off-by: Rune Morling <[email protected]>
@ermo ermo force-pushed the add-worklog.py-sort-option branch from e3c1ee4 to a300869 Compare February 24, 2024 17:35
@ermo ermo requested a review from silkeh February 25, 2024 02:41
Copy link
Member

@silkeh silkeh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@silkeh silkeh merged commit bb245a4 into main Feb 25, 2024
2 checks passed
@EbonJaeger EbonJaeger deleted the add-worklog.py-sort-option branch February 25, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants