[bug#77681,v2,5/6] doc: Emacs Packages: Remove outdated commentary on testing.

Message ID CH3PR84MB342484A4FCFD313F0D654461C5B42@CH3PR84MB3424.NAMPRD84.PROD.OUTLOOK.COM
State New
Headers
Series None |

Commit Message

Morgan Smith April 9, 2025, 9:53 p.m. UTC
  * doc/contributing.texi (Emacs Packages): Removeoutdated commentary on
testing.  We now run the tests by default like is done in other build systems.

Change-Id: I245a0a412615a1177fbd1b6e723fda3749f7e49f
---
 doc/contributing.texi | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)
  

Comments

Ludovic Courtès April 14, 2025, 3 p.m. UTC | #1
Morgan Smith <Morgan.J.Smith@outlook.com> writes:

> * doc/contributing.texi (Emacs Packages): Removeoutdated commentary on
> testing.  We now run the tests by default like is done in other build systems.
>
> Change-Id: I245a0a412615a1177fbd1b6e723fda3749f7e49f

Applied, thanks!
  
Morgan Smith April 14, 2025, 4:21 p.m. UTC | #2
Ludovic Courtès <ludo@gnu.org> writes:

> Morgan Smith <Morgan.J.Smith@outlook.com> writes:
>
>> * doc/contributing.texi (Emacs Packages): Removeoutdated commentary on
>> testing.  We now run the tests by default like is done in other build systems.
>>
>> Change-Id: I245a0a412615a1177fbd1b6e723fda3749f7e49f
>
> Applied, thanks!

Please consider reverting this.

Sorry for the inconvenience!

This was meant was the emacs-team branch (and is already committed
there).  The 0th patch had emacs-team in the subject.  I should figure
out the command line invocation to add that to all patches.

I'm not sure if this will cause merge conflicts.  If git is smart enough
then maybe we can just leave it as is.  I'm not sure how that process
works.
  
Ludovic Courtès April 14, 2025, 6:32 p.m. UTC | #3
Morgan Smith <morgan.j.smith@outlook.com> writes:

> Please consider reverting this.
>
> Sorry for the inconvenience!
>
> This was meant was the emacs-team branch (and is already committed
> there).  The 0th patch had emacs-team in the subject.  I should figure
> out the command line invocation to add that to all patches.

Oh, apologies; I was Cc'd on just this one and hadn’t realized this was
part of a series.

I’ll revert it shortly.

Thanks,
Ludo’.
  

Patch

diff --git a/doc/contributing.texi b/doc/contributing.texi
index ab4f30d54b..eb105fc6fb 100644
--- a/doc/contributing.texi
+++ b/doc/contributing.texi
@@ -1433,14 +1433,7 @@  Emacs Packages
 Emacs packages should preferably use the Emacs build system
 (@pxref{emacs-build-system}), for uniformity and the benefits provided
 by its build phases, such as the auto-generation of the autoloads file
-and the byte compilation of the sources.  Because there is no
-standardized way to run a test suite for Emacs packages, tests are
-disabled by default.  When a test suite is available, it should be
-enabled by setting the @code{#:tests?} argument to @code{#true}.  By
-default, the command to run the test is @command{make check}, but any
-command can be specified via the @code{#:test-command} argument.  The
-@code{#:test-command} argument expects a list containing a command and
-its arguments, to be invoked during the @code{check} phase.
+and the byte compilation of the sources.
 
 The Elisp dependencies of Emacs packages are typically provided as
 @code{propagated-inputs} when required at run time.  As for other