On Sat, 25 Jan 2025 at 02:02, <curc...@apache.org> wrote:
>
> This is an automated email from the ASF dual-hosted git repository.
>
> curcuru pushed a commit to branch master
> in repository https://gitbox.apache.org/repos/asf/whimsy.git
>
>
> The following commit(s) were added to refs/heads/master by this push:
>      new d9f365bc Doesn't seem needed
> d9f365bc is described below
>
> commit d9f365bc2198d73c876f30fbe6f4194f9d1b33a9
> Author: Shane Curcuru <a...@shanecurcuru.org>
> AuthorDate: Fri Jan 24 21:01:56 2025 -0500
>
>     Doesn't seem needed
> ---
>  lib/whimsy/asf/member-files.rb | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/lib/whimsy/asf/member-files.rb b/lib/whimsy/asf/member-files.rb
> index 5ce45982..96436c36 100644
> --- a/lib/whimsy/asf/member-files.rb
> +++ b/lib/whimsy/asf/member-files.rb
> @@ -181,7 +181,6 @@ module ASF
>      def self.sort_board_nominees(contents, entries=nil)
>        sections = contents.split(%r{^-{10,}\n})
>        header = sections.shift(2)
> -      sections.pop if sections.last.strip == ''

-1

AFAICT, this trims trailing blanks, and should be kept unless proved otherwise.

>        sections.append(*entries) if entries # add new entries if any
>        names = {}
>        # replace 'each' by 'sort_by!' to sort by last name
>

Reply via email to