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

Replace com.google.common.base.MoreObjects.firstNonNull & com.google.common.base.Preconditions.checkArgument by Java alternatives #2228

Merged
merged 12 commits into from
Jul 5, 2023

Conversation

pks-1981
Copy link
Contributor

@pks-1981 pks-1981 commented Jun 19, 2023

Proposal. Replacing com.google.common.base.MoreObjects.firstNonNull & com.google.common.base.Preconditions.checkArgument

References

#2111

Submitter checklist

  • The PR request is well described and justified, including the body and the references
  • The PR title represents the desired changelog entry
  • The repository's code style is followed (see the contributing guide)
  • Test coverage that demonstrates that the change works as expected
  • For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org

Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

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

Requesting changes because of API changes that do not seem to be related to the PRs purpose, or justified otherwise. Let's do them separately at least. There is also a bunch of other refactorings, but I do not mind

@@ -27,7 +27,7 @@ public abstract class AbstractScenarios implements Scenarios {

private final ScenariosStore store;

public AbstractScenarios(ScenariosStore store) {
protected AbstractScenarios(ScenariosStore store) {
Copy link
Member

Choose a reason for hiding this comment

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

Revert? Seems to be a legit API

@@ -48,7 +48,7 @@ public abstract class AbstractStubMappings implements StubMappings {
protected final List<StubLifecycleListener> stubLifecycleListeners;
protected final StubMappingStore store;

public AbstractStubMappings(
protected AbstractStubMappings(
Copy link
Member

Choose a reason for hiding this comment

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

Legit API?

@pks-1981
Copy link
Contributor Author

pks-1981 commented Jul 5, 2023

@oleg-nenashev
Hello
I don't understand...
I did everything, but it's like nothing...
image

@tomakehurst tomakehurst requested a review from a team as a code owner July 5, 2023 09:25
@tomakehurst tomakehurst merged commit f729b52 into wiremock:master Jul 5, 2023
6 of 7 checks passed
@pks-1981 pks-1981 deleted the parameter branch July 5, 2023 12:08
@oleg-nenashev oleg-nenashev changed the title Proposal. Replacing com.google.common.base.MoreObjects.firstNonNull &… Replace com.google.common.base.MoreObjects.firstNonNull & com.google.common.base.Preconditions.checkArgument by Java alternatives Jul 5, 2023
@oleg-nenashev oleg-nenashev added the chore Maintenance label Jul 5, 2023
@oleg-nenashev
Copy link
Member

@pks-1981

I don't understand...
I did everything, but it's like nothing...

Is it still actual?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants