This is an automated email from the ASF dual-hosted git repository.

sebb 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 47a632ad fix: board minutes collation: no longer try to reuse year 
(#275)
47a632ad is described below

commit 47a632add52443526ead7d90d52b73d64aead0e4
Author: Arnout Engelen <arn...@bzzt.net>
AuthorDate: Mon Jun 16 18:14:39 2025 +0200

    fix: board minutes collation: no longer try to reuse year (#275)
    
    previously, this script fetched 
https://www.apache.org/foundation/board/calendar.html
    to re-use its style and the 'list of years' paragraph.
    
    However, presumably due to changes to calendar.html, this has been showing a
    random different paragraph on the minutes pages, currently 'Meeting times 
vary'.
    
    Instead of being clever, I don't think we have a strong need to have that 
year
    paragraph here, so we'd be better off just replacing the main content 
wholesale.
---
 tools/collate_minutes.rb | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/tools/collate_minutes.rb b/tools/collate_minutes.rb
index c867f4cf..7e428eb5 100755
--- a/tools/collate_minutes.rb
+++ b/tools/collate_minutes.rb
@@ -790,13 +790,7 @@ def layout(title = nil)
 #   $calendar.at('h2').content = "Board Meeting Minutes"
   end
 
-  # Adjust the page header
-
-  # find the intro para; assume it is the first para with a strong tag
-  # then back up to the main container class for the page content
-  section = $calendar.at('.container p strong').parent.parent
-  # Extract all the paragraphs
-  paragraphs = section.search('p')
+  section = $calendar.at('#maincontent .container')
 
   # remove all the existing content
   section.children.each {|child| child.remove}
@@ -831,10 +825,6 @@ def layout(title = nil)
     end
   }
 
-  # and the second para which is assumed to be the list of years
-  section.add_child paragraphs[1]
-  section.add_child "\n" # separator to make it easier to read source
-
   # now add the content provided by the builder block
   content.at('body').children.each {|child| section.add_child child}
 

Reply via email to