★ wanayoo — archive 1999 https://github.com/nodejs/node-core-utils/pull/329

Nouvelle recherche | Portail wanayoo

nodejs / node-core-utils

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tweak CI walk filter order #329

Closed

refack wants to merge 1 commit into master from ci-walk-parse-order

Member

refack commented Mar 22, 2019

Moved more specific pattern higher up


          Tweak CI walk filter order

8a6c8e0

refack mentioned this pull request

CI failures: 20190322 nodejs/reliability#21

Closed

refack self-assigned this

joyeecheung reviewed

lib/ci/ci_failure_parser.js

  
      @@ -238,23 +236,25 @@ const FAILURE_FILTERS = [{
    
        }, {

          filter(ctx, text) {

            const patterns = [{

              pattern: /FATAL: .+/g,

              context: { index: -1, contextBefore: 0, contextAfter: 5 }

              pattern: /There is not enough space in the file system./g,

joyeecheung Mar 23, 2019
Member

The ones moved here are actually more specific? Can you split those instead?

refack Mar 23, 2019
Author Member

Yeah I wrote this patch on GitHub, hoping I hit it in one try ¯\(ツ)/¯
I need to dig into this locally a bit more...

joyeecheung added the Work In Progress label

Member

BridgeAR commented May 30, 2020

Contributor

github-actions bot commented Aug 29, 2020

This PR is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made.

github-actions bot added the stale label

codebytere closed this

codebytere deleted the ci-walk-parse-order branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

You can’t perform that action at this time.