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

Extend the Field utility with presentIn(...) and isNested() methods #498

Merged
merged 1 commit into from
Nov 15, 2019

Conversation

dmitrykuzmin
Copy link
Contributor

@dmitrykuzmin dmitrykuzmin commented Nov 13, 2019

This PR extends the Field with some utilities needed by the core-java#1197.

Spine base version advances to 1.2.3.

@dmitrykuzmin
Copy link
Contributor Author

@armiol, @dmdashenkov PTAL.

@codecov
Copy link

codecov bot commented Nov 13, 2019

Codecov Report

Merging #498 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master    #498      +/-   ##
===========================================
+ Coverage     72.79%   72.8%   +0.01%     
- Complexity     2608    2611       +3     
===========================================
  Files           454     454              
  Lines         10560   10566       +6     
  Branches        612     613       +1     
===========================================
+ Hits           7687    7693       +6     
  Misses         2697    2697              
  Partials        176     176

Copy link
Collaborator

@armiol armiol left a comment

Choose a reason for hiding this comment

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

@dmitrykuzmin LGTM. However, can you please rename the PR (and maybe tweak the description), so that it makes sense in terms of base. There is no client here.

@dmitrykuzmin dmitrykuzmin changed the title Validate the subscription topic on client Extend the Field utility with presentIn(...) and isNested() methods Nov 15, 2019
@dmitrykuzmin dmitrykuzmin merged commit 585832b into master Nov 15, 2019
@dmitrykuzmin dmitrykuzmin deleted the validate-client-subscriptions branch November 15, 2019 16:07
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