unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template):
       [not found] <E1Vj5RK-0003OM-4J@vcs.savannah.gnu.org>
@ 2013-11-21 15:41 ` Glenn Morris
  2013-11-21 22:51   ` Bozhidar Batsov
  0 siblings, 1 reply; 9+ messages in thread
From: Glenn Morris @ 2013-11-21 15:41 UTC (permalink / raw)
  To: Bozhidar Batsov; +Cc: emacs-devel


Hi,

Please add a ChangeLog entry at the appropriate date for this and other
recent changes (to test). Also, defcustom with non-trivial changes
should get :version bumps (don't know if this is non-trivial).

Seems lke test ruby-exit!-font-lock should have expected-result :failed
for now, else it causes a coverage failure on hydra, which could mask
the introduction of real problems.

See eg http://hydra.nixos.org/build/6852191



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template):
  2013-11-21 15:41 ` trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template): Glenn Morris
@ 2013-11-21 22:51   ` Bozhidar Batsov
  2013-11-21 23:07     ` Glenn Morris
  0 siblings, 1 reply; 9+ messages in thread
From: Bozhidar Batsov @ 2013-11-21 22:51 UTC (permalink / raw)
  To: Glenn Morris; +Cc: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 1026 bytes --]

Seems odd to include such trivial changes in the changelog, as nobody is
likely to be interested in reading about them. This particular defcustom
was recently introduced, that's why I considered altering its default an
extremely tiny change.

As for the tests - I'll change the state of the problematic test to have
expected-result :failed. Sorry about that. Btw, what's the value of listing
test changes in the changelog? Seems to me that pretty few people would be
interested in reading about such changes.


On 21 November 2013 17:41, Glenn Morris <rgm@gnu.org> wrote:

>
> Hi,
>
> Please add a ChangeLog entry at the appropriate date for this and other
> recent changes (to test). Also, defcustom with non-trivial changes
> should get :version bumps (don't know if this is non-trivial).
>
> Seems lke test ruby-exit!-font-lock should have expected-result :failed
> for now, else it causes a coverage failure on hydra, which could mask
> the introduction of real problems.
>
> See eg http://hydra.nixos.org/build/6852191
>

[-- Attachment #2: Type: text/html, Size: 1468 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template):
  2013-11-21 22:51   ` Bozhidar Batsov
@ 2013-11-21 23:07     ` Glenn Morris
  2013-11-21 23:17       ` Glenn Morris
  0 siblings, 1 reply; 9+ messages in thread
From: Glenn Morris @ 2013-11-21 23:07 UTC (permalink / raw)
  To: Bozhidar Batsov; +Cc: emacs-devel

Bozhidar Batsov wrote:

> Seems odd to include such trivial changes in the changelog, as nobody is
> likely to be interested in reading about them.

It is the Emacs convention to record everything (essentially) in
ChangeLogs. It's no extra work. Eg, use `C-x 4 a' to create the
ChangeLog entry, then use log-edit-insert-changelog to create the VCS
log entry from the ChangeLog.

> This particular defcustom was recently introduced,

Then it should definitely have a :version tag. :)

(BTW, there isn't even a ChangeLog entry for the introduction.)



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template):
  2013-11-21 23:07     ` Glenn Morris
@ 2013-11-21 23:17       ` Glenn Morris
  2013-11-22 12:38         ` Bozhidar Batsov
  0 siblings, 1 reply; 9+ messages in thread
From: Glenn Morris @ 2013-11-21 23:17 UTC (permalink / raw)
  To: Bozhidar Batsov; +Cc: emacs-devel

Glenn Morris wrote:

> Then it should definitely have a :version tag. :)

And to further the pedantry, the first line of the doc-string should be
a complete sentence. :)



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template):
  2013-11-21 23:17       ` Glenn Morris
@ 2013-11-22 12:38         ` Bozhidar Batsov
  2013-11-23  2:37           ` Glenn Morris
  0 siblings, 1 reply; 9+ messages in thread
From: Bozhidar Batsov @ 2013-11-22 12:38 UTC (permalink / raw)
  To: Glenn Morris; +Cc: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 327 bytes --]

Done. I saw you had already fixed the test expectation. Thanks for the
feedback!


On 22 November 2013 01:17, Glenn Morris <rgm@gnu.org> wrote:

> Glenn Morris wrote:
>
> > Then it should definitely have a :version tag. :)
>
> And to further the pedantry, the first line of the doc-string should be
> a complete sentence. :)
>

[-- Attachment #2: Type: text/html, Size: 661 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template):
  2013-11-22 12:38         ` Bozhidar Batsov
@ 2013-11-23  2:37           ` Glenn Morris
  2013-11-23 13:23             ` Bozhidar Batsov
  0 siblings, 1 reply; 9+ messages in thread
From: Glenn Morris @ 2013-11-23  2:37 UTC (permalink / raw)
  To: Bozhidar Batsov; +Cc: emacs-devel


Thanks for adding a test/ChangeLog entry.
Do please list the names of the tests in the entry.
ChangeLog entries should name every function, variable that they touch.
I notice this did not happen eg in
http://lists.gnu.org/archive/html/emacs-diffs/2013-11/msg00216.html
which changed an option, added several new functions, and modified
another, but only mentioned the last in the ChangeLog.

Similarly
http://lists.gnu.org/archive/html/emacs-diffs/2013-11/msg00150.html
does not list the two new options as entities in their own right.

Please could you review all your past changes with an eye to that,
and improve the ChangeLogs where necessary?

Also, maybe you want to add a NEWS entry about those new options.
This need be nothing more than:

  ** New Ruby mode options: foo, bar.

in the "Changes in Specialized Modes and Packages in Emacs 24.4" section.



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template):
  2013-11-23  2:37           ` Glenn Morris
@ 2013-11-23 13:23             ` Bozhidar Batsov
  2013-11-25 19:40               ` Glenn Morris
  0 siblings, 1 reply; 9+ messages in thread
From: Bozhidar Batsov @ 2013-11-23 13:23 UTC (permalink / raw)
  To: Glenn Morris; +Cc: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 1261 bytes --]

On 23 November 2013 04:37, Glenn Morris <rgm@gnu.org> wrote:

>
> Thanks for adding a test/ChangeLog entry.
> Do please list the names of the tests in the entry.
>

Will do.


> ChangeLog entries should name every function, variable that they touch.
> I notice this did not happen eg in
> http://lists.gnu.org/archive/html/emacs-diffs/2013-11/msg00216.html
> which changed an option, added several new functions, and modified
> another, but only mentioned the last in the ChangeLog.
>

Since the new functions were extracted from the modified function I wasn't
sure if they needed mentioning.
Basically in a situation like this I should list all symbols like this
(sym1, sym2...): and write a unified summary regarding
all of them, right?


>
> Similarly
> http://lists.gnu.org/archive/html/emacs-diffs/2013-11/msg00150.html
> does not list the two new options as entities in their own right.
>
> Please could you review all your past changes with an eye to that,
> and improve the ChangeLogs where necessary?
>

Sure.


>
> Also, maybe you want to add a NEWS entry about those new options.
> This need be nothing more than:
>
>   ** New Ruby mode options: foo, bar.
>

Will do.


>
> in the "Changes in Specialized Modes and Packages in Emacs 24.4" section.
>

[-- Attachment #2: Type: text/html, Size: 2444 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template):
  2013-11-23 13:23             ` Bozhidar Batsov
@ 2013-11-25 19:40               ` Glenn Morris
  2013-11-26  6:47                 ` Bozhidar Batsov
  0 siblings, 1 reply; 9+ messages in thread
From: Glenn Morris @ 2013-11-25 19:40 UTC (permalink / raw)
  To: Bozhidar Batsov; +Cc: emacs-devel

Bozhidar Batsov wrote:

> Since the new functions were extracted from the modified function I wasn't
> sure if they needed mentioning.
> Basically in a situation like this I should list all symbols like this
> (sym1, sym2...): and write a unified summary regarding
> all of them, right?

I'd write something like:

(newfunc1, newfunc2): New functions, extracted from oldfunc.
(oldfunc): Split into several functions, to <do new stuff>.



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template):
  2013-11-25 19:40               ` Glenn Morris
@ 2013-11-26  6:47                 ` Bozhidar Batsov
  0 siblings, 0 replies; 9+ messages in thread
From: Bozhidar Batsov @ 2013-11-26  6:47 UTC (permalink / raw)
  To: Glenn Morris; +Cc: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 579 bytes --]

Thanks for the suggestion. I'll do the updates today.


On 25 November 2013 21:40, Glenn Morris <rgm@gnu.org> wrote:

> Bozhidar Batsov wrote:
>
> > Since the new functions were extracted from the modified function I
> wasn't
> > sure if they needed mentioning.
> > Basically in a situation like this I should list all symbols like this
> > (sym1, sym2...): and write a unified summary regarding
> > all of them, right?
>
> I'd write something like:
>
> (newfunc1, newfunc2): New functions, extracted from oldfunc.
> (oldfunc): Split into several functions, to <do new stuff>.
>

[-- Attachment #2: Type: text/html, Size: 960 bytes --]

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2013-11-26  6:47 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <E1Vj5RK-0003OM-4J@vcs.savannah.gnu.org>
2013-11-21 15:41 ` trunk r115158: * lisp/progmodes/ruby-mode.el (ruby-custom-encoding-magic-comment-template): Glenn Morris
2013-11-21 22:51   ` Bozhidar Batsov
2013-11-21 23:07     ` Glenn Morris
2013-11-21 23:17       ` Glenn Morris
2013-11-22 12:38         ` Bozhidar Batsov
2013-11-23  2:37           ` Glenn Morris
2013-11-23 13:23             ` Bozhidar Batsov
2013-11-25 19:40               ` Glenn Morris
2013-11-26  6:47                 ` Bozhidar Batsov

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).