Repository navigation
Simplify+optimize - #856
Conversation
use simpler constructs, avoid calls to external apps, compact some statements to reduce debugging (-x) output
Also remove exports only performed with --quiet; preparing for `eval`
Could be installed but not configured
Set trap to ensure lock removal; warn if cache locked
less -x output too
use grep -e
|
Hm, I got Error: Resource not accessible by integration... Not sure what that means |
|
Thanks for this excellent and instructive work. Don't worry about the error you mentioned: the actions seem to be broken lately and need some attention and revision. |
| # The GitHub API is rate limit has likely been reached | ||
| if [ "${METHOD}" == github ] && [ ! -s "${CACHE_FILE}" ]; then | ||
| if [ "${ACTION}" != "prettylist" ] && [ "${ACTION}" != "remove" ] && [ "${ACTION}" != "purge" ]; then | ||
| if ! [[ ' prettylist remove purge ' =~ " ${ACTION} " ]]; then |
philclifford
left a comment
There was a problem hiding this comment.
LGTM
Limited operational testing so far
|
Looking at things I rarely use part 1: |
Yup — last commit (1f4581b), which purported to fix |
Hi, I've found
deb-getreally useful and played with it for a bit.I've tried to optimize performance (by avoiding repeated external calls and caching some results — e.g.
listis about 4 times faster), fix some (minor) bugs and simplify some shell constructs (e.g. replacinggrep | cutwithsed ...), fix someshellcheckwarnings etc.I've tried to make the diffs between commits as small / clear as possible (as opposed to a big rewrite), hence the number of commits.
I've tested it as much as I could on my normal workflows, hopefully github-actions can also help here (I'm not sure how to run testing on my fork).