Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

fix(update): Skip logs starting with (chore) #4800

Merged
merged 2 commits into from
Mar 10, 2022
Merged

fix(update): Skip logs starting with (chore) #4800

merged 2 commits into from
Mar 10, 2022

Conversation

rashil2000
Copy link
Member

Description

Skip logs starting with (chore) instead of chore

Motivation and Context

Closes #4799

How Has This Been Tested?

image

Checklist:

  • I have read the Contributing Guide.
  • I have updated the documentation accordingly.
  • I have updated the tests accordingly.

@niheaven
Copy link
Member

niheaven commented Mar 10, 2022 •
edited
Loading

Arg in --grep needn't be escaped, right?

@rashil2000
Copy link
Member Author

No (see screenshot)

@niheaven
Copy link
Member

OK, LGTM, add changelog entry and could be merged.

@rashil2000 rashil2000 merged commit 476b507 into develop Mar 10, 2022
@rashil2000 rashil2000 deleted the rashil2000-patch-1 branch March 10, 2022 07:44
@niheaven niheaven mentioned this pull request May 3, 2022
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rashil2000 @niheaven