From: Emanuel Berg <incal@dataswamp.org>
To: emacs-devel@gnu.org
Subject: Re: [NonGNU ELPA] Package suggestion: yeetube
Date: Wed, 04 Oct 2023 15:27:30 +0200 [thread overview]
Message-ID: <87mswyecvx.fsf@dataswamp.org> (raw)
In-Reply-To: 87ttr67rkt.fsf@thanosapollo.com
Thanos Apollo wrote:
> I've been trying to learn elisp and made this simple niche
> package, to watch & download youtube content via Emacs using
> only free software.
>
> Homepage: https://git.thanosapollo.com/yeetube/about/
Nice!
I did
(checkdoc-current-buffer t)
and also byte compiled with this command
$ emacs -Q \
--batch \
--eval "(setq load-path (append load-path (list \".\")))" \
-f batch-byte-compile *.el
to discover the following stuff you can improve:
*** yeetube-buffer.el: checkdoc-current-buffer
yeetube-buffer.el:185: Arguments occur in the doc string out
of order
In yeetube-buffer-create:
yeetube-buffer.el:209:8: Warning: ‘backward-delete-char’ is
for interactive use only; use ‘delete-char’ instead.
In yeetube-download-videos:
yeetube.el:250:2: Warning: docstring has wrong usage of
unescaped single quotes (use \=' or different quoting such as
`...')
In end of data:
yeetube.el:275:9: Warning: the function
‘yeetube-change-download-audio-format’ is not known to
be defined.
yeetube.el:274:9: Warning: the function
‘yeetube-change-download-directory’ is not known to
be defined.
PS. If you are into automated downloading, check out these files
https://dataswamp.org/~incal/conf/.zsh/dl
https://dataswamp.org/~incal/conf/.zsh/yt-dlp
https://dataswamp.org/~incal/#mpv
It doesn't have an interface tho like yours, it just
downloads bulk and then you remove stuff you don't like
from mpv.
GLHF!
--
underground experts united
https://dataswamp.org/~incal
next prev parent reply other threads:[~2023-10-04 13:27 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-04 7:24 [NonGNU ELPA] Package suggestion: yeetube Thanos Apollo
2023-10-04 8:39 ` Stefan Kangas
2023-10-04 8:48 ` Thanos Apollo
2023-10-04 12:50 ` Stefan Kangas
2023-10-04 13:37 ` Thanos Apollo
2023-10-05 10:24 ` Richard Stallman
2023-10-05 10:44 ` Thanos Apollo
2023-10-05 11:07 ` Ihor Radchenko
2024-02-11 7:43 ` Using tor & url Thanos Apollo
2023-10-04 13:14 ` [NonGNU ELPA] Package suggestion: yeetube Daniel Martín
2023-10-04 13:33 ` Thanos Apollo
2023-10-04 15:22 ` Philip Kaludercic
2023-10-04 15:35 ` [External] : " Drew Adams
2023-10-04 16:26 ` Thanos Apollo
2023-10-06 8:59 ` Philip Kaludercic
2023-10-04 15:37 ` Thanos Apollo
2023-10-04 13:27 ` Emanuel Berg [this message]
2023-10-05 2:15 ` Thanos Apollo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87mswyecvx.fsf@dataswamp.org \
--to=incal@dataswamp.org \
--cc=emacs-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).