Wikipedia:Bots/Requests for approval/ZackBot 8

The following discussion is an archived debate. Please do not modify it. To request review of this BRFA, please start a new section at WT:BRFA. The result of the discussion was

Approved.

ZackBot 8

Operator: Zackmann08 (talk · contribs · SUL · edit count · logs · page moves · block log · rights log · ANI search)

Time filed: 20:49, Thursday, January 5, 2017 (UTC)

Automatic, Supervised, or Manual: Automatic

Programming language(s): Ruby

Source code available: User:ZackBot/NHL

Function overview: To allow for Template:Infobox NHL team season to be upmerged to Template:Infobox ice hockey team season, a single param needs to be added to all NHL team season occurrences... League = NHL

Links to relevant discussions (where appropriate): Wikipedia:Templates_for_discussion/Log/2017_January_5#Template:Infobox_KHL_team_season

Edit period(s): One time run

Estimated number of pages affected: 1500

Exclusion compliant (Yes/No): yes

Already has a bot flag (Yes/No): Yes

Function details: This will go through all NHL team pages that use Template:Infobox NHL team season. It will locate the "Season" param, which is presumably at the top of the template, and insert League = NHL before it. I have also done my best to preserve the indentation patterns that already exist in the template.

Discussion

Looks good so far, but I'd like to wait on Wikipedia:Templates_for_discussion/Log/2017_January_5#Template:Infobox_KHL_team_season, otherwise this seems like it would be a bunch of redundant, unnecessary edits for a merge that theoretically might not happen. It probably won't be an issue, though. :P --slakr^\ talk / 02:14, 7 January 2017 (UTC)[reply]

@Slakr: the merge has happened. There was some confusion... Basically I initially made the merge, then it was reverted by a user who said I needed to discuss it... So I took it to a WP:TFD and they said "Oh I didn't mean a TFD... I just meant talk about it. I have no problem with it as long as it works!" So the merge was re-done... --Zackmann08 (^{Talk to me}/_{What I been doing}) 05:06, 8 January 2017 (UTC)[reply]

Approved for trial (50 edits). Please provide a link to the relevant contributions and/or diffs when the trial is complete. — MusikAnimal ^talk 02:47, 16 January 2017 (UTC)[reply]

Trial complete. @MusikAnimal: no issues that I could see. --Zackmann08 (^{Talk to me}/_{What I been doing}) 18:17, 16 January 2017 (UTC)[reply]

Approved. @Zackmann08: Only thing is again, it is highly preferred these one-time, mass changes have a link to the relevant discussion in the edit summary. I would link back to this BRFA, as the TfD by itself seems a bit odd given you opened and closed it yourself :) — MusikAnimal ^talk 18:26, 16 January 2017 (UTC)[reply]

@MusikAnimal: thanks for the reminder about linking to the BRFA. Made that change. --Zackmann08 (^{Talk to me}/_{What I been doing}) 19:37, 16 January 2017 (UTC)[reply]

The above discussion is preserved as an archive of the debate. Please do not modify it. To request review of this BRFA, please start a new section at WT:BRFA.