Skip to content

Conversation

@ihsandemir
Copy link
Collaborator

fixes #612

@ihsandemir ihsandemir added this to the 4.0 milestone Oct 16, 2020
@ihsandemir ihsandemir self-assigned this Oct 16, 2020
@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

sancar
sancar previously approved these changes Oct 26, 2020
*
* @return the version of the client
*/
static const constexpr boost::string_view &version() {
Copy link
Contributor

Choose a reason for hiding this comment

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

constexpr already means const.

Suggested change
static const constexpr boost::string_view &version() {
static constexpr boost::string_view &version() {

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@ihsandemir ihsandemir merged commit 3b98512 into hazelcast:master Oct 27, 2020
@andysCaplin andysCaplin mentioned this pull request Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Version API

4 participants