Skip to content

Update expectations after random name generator upgrade#270

Merged
greg-at-moderne merged 1 commit intomainfrom
greg-random-name-fix
Dec 18, 2025
Merged

Update expectations after random name generator upgrade#270
greg-at-moderne merged 1 commit intomainfrom
greg-random-name-fix

Conversation

@greg-at-moderne
Copy link
Contributor

What's changed?

Changing test expectations in PrependRandomNameTest.
The library got a new version released yesterday. I haven't investigated, but they might have changed either the dictionary, the random seed, or the randomization algorithm.
It still seems to behave deterministically.

What's your motivation?

Fix broken CI.

@greg-at-moderne greg-at-moderne self-assigned this Dec 18, 2025
@github-project-automation github-project-automation bot moved this to In Progress in OpenRewrite Dec 18, 2025
@greg-at-moderne greg-at-moderne merged commit de332fa into main Dec 18, 2025
2 checks passed
@greg-at-moderne greg-at-moderne deleted the greg-random-name-fix branch December 18, 2025 08:53
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenRewrite Dec 18, 2025
mergify bot added a commit to robfrank/linklift that referenced this pull request Jan 9, 2026
…rom 3.20.0 to 3.21.0 [skip ci]

Bumps [org.openrewrite.recipe:rewrite-logging-frameworks](https://github.com/openrewrite/rewrite-logging-frameworks) from 3.20.0 to 3.21.0.
Release notes

*Sourced from [org.openrewrite.recipe:rewrite-logging-frameworks's releases](https://github.com/openrewrite/rewrite-logging-frameworks/releases).*

> 3.21.0
> ------
>
> What's Changed
> --------------
>
> * Update expectations after random name generator upgrade by [`@​greg-at-moderne`](https://github.com/greg-at-moderne) in [openrewrite/rewrite-logging-frameworks#270](https://github.com/openrewrite/rewrite-logging-frameworks/pull/270)
> * Remove vulnerable dependency from test scope by [`@​Laurens-W`](https://github.com/Laurens-W) in [openrewrite/rewrite-logging-frameworks#271](https://github.com/openrewrite/rewrite-logging-frameworks/pull/271)
>
> **Full Changelog**: <openrewrite/rewrite-logging-frameworks@v3.20.0...v3.21.0>


Commits

* [`b957893`](openrewrite/rewrite-logging-frameworks@b957893) Move suppressions to here from rewrite-build-gradle-plugin as it only applies...
* [`7acc41c`](openrewrite/rewrite-logging-frameworks@7acc41c) Remove vulnerable dependency from test scope ([#271](https://github.com/openrewrite/rewrite-logging-frameworks/issues/271))
* [`f29a637`](openrewrite/rewrite-logging-frameworks@f29a637) Pick the correct example recipe
* [`34e398e`](openrewrite/rewrite-logging-frameworks@34e398e) Update documentation examples
* [`7b46a16`](openrewrite/rewrite-logging-frameworks@7b46a16) Update recipe documentation examples
* [`89f85a8`](openrewrite/rewrite-logging-frameworks@89f85a8) Generate a recipes.csv for the recipes defined in this repository
* [`5e648a7`](openrewrite/rewrite-logging-frameworks@5e648a7) OpenRewrite recipe best practices
* [`de332fa`](openrewrite/rewrite-logging-frameworks@de332fa) Update expectations after random name generator upgrade ([#270](https://github.com/openrewrite/rewrite-logging-frameworks/issues/270))
* See full diff in [compare view](openrewrite/rewrite-logging-frameworks@v3.20.0...v3.21.0)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=org.openrewrite.recipe:rewrite-logging-frameworks&package-manager=maven&previous-version=3.20.0&new-version=3.21.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
  
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot show  ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant