bug#41086: Fwd: bug#41086: [PATCH] Add user-defined column widths to package-list (package.el)

[Stefan Kangas <stefankangas@gmail.com>]

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

---------- Forwarded message ---------
Från: Chris McMahan <cmcmahan@gmail.com>
Date: tis 5 maj 2020 kl 03:10
Subject: Re: bug#41086: [PATCH] Add user-defined column widths to
package-list (package.el)
To: Stefan Kangas <stefankangas@gmail.com>


Hello Stephan,

Revised patch file is attached.

- Chris

On Mon, May 4, 2020 at 6:13 PM Stefan Kangas <stefankangas@gmail.com> wrote:
>
> Hi Chris,
>
> Chris McMahan <cmcmahan@gmail.com> writes:
>
> > Yes, this is my first patch.
>
> Welcome to Emacs!  Let's hope we see more contributions from you in
> the future.
>
> > I have no problem signing the copyright. I just need some guidance on the
> > process.
>
> OK, great.  Eli, could you please help Chris get started with the
> process?
>
> > The changelog entry follows:
> >
> > * package.el (package-menu-mode): User can now adjust the column widths of
> > the package menu by setting the values of the following defcustoms.
> >
> > (package-name-column-width) New defcustom
> > (package-version-column-width) New defcustom
> > (package-status-column-width) New defcustom
>
> Thank you, but could you add this as part of the commit message?  See
> the file CONTRIBUTE for more, or previous commits in the git repository.
>
> > Subject: [PATCH] Allow user to adjust column widths in the package menu
>
> Maybe better: "Customizable column widths in package menu"
>
> Other than that, looks good to me.  We just have to wait for your
> copyright assignment to come through before we can push it.
>
> Thanks again.
>
> Best regards,
> Stefan Kangas



-- 
Chris McMahan <cmcmahan@gmail.com>

[-- Attachment #2: 0001-Allow-user-to-adjust-column-widths-in-the-package-me.patch --]
[-- Type: application/x-patch, Size: 1962 bytes --]