style: Use grep and map only in block form

[perlpunk]

• https://metacpan.org/pod/Perl::Critic::Policy::BuiltinFunctions::RequireBlockGrep
• https://metacpan.org/pod/Perl::Critic::Policy::BuiltinFunctions::RequireBlockMap

See also #7056

[perlpunk]

https://app.circleci.com/pipelines/github/os-autoinst/openQA/19721/workflows/b80c43c7-8819-4d0f-bc32-7133c828461f/jobs/196777

Commit b0529f4ea3:
6: B1 Line exceeds max length (81>80): "https://metacpan.org/pod/Perl::Critic::Policy::BuiltinFunctions::RequireBlockGrep"
make: *** [Makefile:395: test-gitlint] Error 1

Am I supposed to break a URL into two pieces because of gitlint?

[perlpunk]

https://app.circleci.com/pipelines/github/os-autoinst/openQA/19721/workflows/b80c43c7-8819-4d0f-bc32-7133c828461f/jobs/196772

    #   Failed test 'exactly one collapse button present'
    #   at t/ui/10-tests_overview.t line 213.
    #          got: '0'
    #     expected: '1'
    # Looks like you failed 1 test of 32.

#   Failed test 'stacking of parallel children'
#   at t/ui/10-tests_overview.t line 224.
# Looks like you failed 1 test of 38.
[15:47:01] t/ui/10-tests_overview.t ................... 
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/38 subtests 

but that test passes for me locally. Retriggering.

[perlpunk]

It's not urgent, but how can I avoid the gitlint error?

[perlpunk]

Also, I sometimes add log lines to a commit message, and having to wrap those is not always a good idea.

[okurz]

It's not urgent, but how can I avoid the gitlint error?

->

• style(gitlint): allow unwrappable longer URLs in git commit message body #7083

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.72%. Comparing base (ead9b8b) to head (51c63e5).
⚠️ Report is 9 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #7078   +/-   ##
=======================================
  Coverage   99.72%   99.72%           
=======================================
  Files         416      416           
  Lines       43076    43078    +2     
=======================================
+ Hits        42956    42958    +2     
  Misses        120      120           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[perlpunk]

@Mergifyio refresh

[mergify]

refresh

✅ Pull request refreshed