Mention Committers On Build Failure#418

Open

AmmarOFA wants to merge 40 commits intojenkinsci:masterfrom

AmmarOFA:email-@wrapping-toggle

Commits on Feb 9, 2026

added a toggle that will fetch user emails and wrap them in <at> tags so the users with extracted emails are tagged

Ammar Zain
authored and
damianszczepanik
committed
added mentionCommitters & mentionDevelopers to jelly.config

Ammar Zain
authored and
damianszczepanik
committed
added updated constructor params to webhook notifier

Ammar Zain
authored and
damianszczepanik
committed
removed extra space in factsbuilder

Ammar Zain
authored and
damianszczepanik
committed
avoiding scope creep, only send mentions on failure notification (defaults to true)

Ammar Zain
authored and
damianszczepanik
committed
got rid of extra constructor and added false defaults to commiters/developers on started cards

Ammar Zain
authored and
damianszczepanik
committed
removed unneeded commons-collection4 dependency

Ammar Zain
authored and
damianszczepanik
committed
Mention failures now extend to unstable and repeated failures. Adjusted mention failure text in jelly.config

Ammar Zain
authored and
damianszczepanik
committed
Mention failure value in jelly now matches value in webhook.java

Ammar Zain
authored and
damianszczepanik
committed
Naming & API clarity for TeamsMentionUtils & tests

Ammar Zain
authored and
damianszczepanik
committed
repeated failure is not a result status therefore does not be included in the shouldMention boolean

Ammar Zain
authored and
damianszczepanik
committed
config.jelly webhook mention toggle now called 'Mention Only When Build Has Failed'

Ammar Zain
authored and
damianszczepanik
committed
removed unnecessary null check in developer fetching

Ammar Zain
authored and
damianszczepanik
committed
unnecessary space removed

Ammar Zain
authored and
damianszczepanik
committed
fixed TeamsMentionUtils & FactsBuilder to use new method/variables names

Ammar Zain
authored and
damianszczepanik
committed
conditional in metionUserOrEmail should use userProperty not prop

Ammar Zain
authored and
damianszczepanik
committed
updated test file variable name from prop to userProperty

Ammar Zain
authored and
damianszczepanik
committed
test now expects 'Unknown user'

Ammar Zain
authored and
damianszczepanik
committed
Removed mentioning developers functionality, now just one toggle for mentioning comitters on failures

Ammar Zain
authored and
damianszczepanik
committed
added json testing for mention feature

Ammar Zain
authored and
damianszczepanik
committed
removed extra space in WebhookBuilder

Ammar Zain
authored and
damianszczepanik
committed
fixed the name of testing json

Ammar Zain
authored and
damianszczepanik
committed
Json should use adaptivecard format not legacy message cards

Ammar Zain
authored and
damianszczepanik
committed
mention sampleIT test cannot use information provided in setup()

Ammar Zain
authored and
damianszczepanik
committed
manual changelogset config for getting emails in the test

Ammar Zain
authored and
damianszczepanik
committed
added missing imports and extracted a helper method for cleaner sampleIT test code

ammzaDal
authored and
damianszczepanik
committed
added helper class ChangeLogSetBuilder to sampleIT imports

ammzaDal
authored and
damianszczepanik
committed
getFullName seems to return null, 0mocking user directly without AffectedFileBuilder

ammzaDal
authored and
damianszczepanik
committed
sorted committers in alphabetical order in completed-failed-with-mentions.json and mocking committers in sampleIT not developers

ammzaDal
authored and
damianszczepanik
committed
added missing set and hashset imports

ammzaDal
authored and
damianszczepanik
committed
added developers to expected json output, ensuring committer order, and making sure developer names are listed properly

ammzaDal
authored and
damianszczepanik
committed
coverting the sorted list into an array before passing it as a changelogset

ammzaDal
authored and
damianszczepanik
committed
now sorting committers in factbuilder

ammzaDal
authored and
damianszczepanik
committed
editing JSON format to match expected output

ammzaDal
authored and
damianszczepanik
committed
more json editing

ammzaDal
authored and
damianszczepanik
committed
updated config.jelly mention message to include unstable builds

ammzaDal
authored and
damianszczepanik
committed
removed trailing space from line 79 in cardbuilder.java

ammzaDal
authored and
damianszczepanik
committed
removed dead code null checks for committers list produced by runwithscm.getCulprits

ammzaDal
authored and
damianszczepanik
committed
Mailer changed to a test dependency

ammzaDal
authored and
damianszczepanik
committed

Commits on Feb 17, 2026

Specified lang3.StringUtils, lang(2) is a banned import
ammzaDal
committed