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

(maint) Add flexibility to logged? to not only match a single line #314

Merged

Conversation

rileynewton
Copy link
Contributor

@rileynewton rileynewton commented May 21, 2024

PR for 3.x for use with 2021.7.x

logged? previously had an undocumented restriction that it would only return true if a single line matched (more than one would return false). This adds the flexibility to remove this restriction, as some log lines may legitimately be repeated.

@rileynewton rileynewton requested a review from a team as a code owner May 21, 2024 20:24
@rileynewton rileynewton changed the title (maint) Add flexibility to logged? to not only match a single line (maint) Add flexibility to logged? to not only match a single line May 21, 2024
@rileynewton rileynewton force-pushed the apply-logged-change-to-3.x branch 4 times, most recently from 29ef728 to 1a1b690 Compare May 21, 2024 20:56
CHANGELOG.md Outdated
@@ -1,3 +1,7 @@
## 4.0.1
Copy link
Contributor

Choose a reason for hiding this comment

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

3.3.2?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Whoops corrected, applied an old stash at first and forgot to change it second time through

@rileynewton rileynewton force-pushed the apply-logged-change-to-3.x branch from 1a1b690 to 60aedaa Compare May 21, 2024 21:15
@rileynewton
Copy link
Contributor Author

Ignoring clj-kondo errors for this backport as tests pass, this will be the last need to use 3.x

@jonathannewman jonathannewman merged commit 0cb21a8 into puppetlabs:3.x May 21, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants