Skip to content
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

Tweak CI walk filter order #329

Closed
wants to merge 1 commit into from
Closed

Tweak CI walk filter order #329

wants to merge 1 commit into from

Conversation

refack
Copy link
Contributor

@refack refack commented Mar 22, 2019

Moved more specific pattern higher up

@refack refack self-assigned this Mar 22, 2019
@@ -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,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 joyeecheung added the Work In Progress PR's that are in progress label Oct 4, 2019
@BridgeAR
Copy link
Member

Ping @refack

@github-actions
Copy link
Contributor

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 github-actions bot added the stale label Aug 29, 2020
@codebytere codebytere closed this Sep 2, 2020
@codebytere codebytere deleted the ci-walk-parse-order branch September 2, 2020 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Work In Progress PR's that are in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants