Skip to content

Cherrypick docs and formatting fixes#247

Merged
brownleej merged 3 commits intoapple:release-5.1from
alecgrieser:cherrypick-docs-and-formatting-fixes
Apr 24, 2018
Merged

Cherrypick docs and formatting fixes#247
brownleej merged 3 commits intoapple:release-5.1from
alecgrieser:cherrypick-docs-and-formatting-fixes

Conversation

@alecgrieser
Copy link
Contributor

These are some fixes to our Java examples and also some go linting fixes that have been merged into master that I think should be pulled into release-5.1.

@alecgrieser alecgrieser requested a review from brownleej April 24, 2018 16:43

// Determines if an API version has already been selected, i.e., if
// APIVersion or MustAPIVersion have already been called.
func IsAPIVersionSelected() bool {
Copy link
Contributor

Choose a reason for hiding this comment

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

Are these functions intended to be added onto the release-5.1 branch, or did they get picked up with another change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, good catch. No, they weren't intended to be added. Let me remove them, but we should make sure they don't disappear when we merge release-5.1 into release-5.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.

Okay, I think the way I did it, it won't cause a problem when we merge, but I guess we'll find out.

@alecgrieser alecgrieser force-pushed the cherrypick-docs-and-formatting-fixes branch from 24f8f9f to 731a8dc Compare April 24, 2018 17:02
Copy link
Contributor

@brownleej brownleej left a comment

Choose a reason for hiding this comment

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

👍

@brownleej brownleej merged commit b651156 into apple:release-5.1 Apr 24, 2018
sfc-gh-jfu pushed a commit to sfc-gh-jfu/foundationdb that referenced this pull request Jul 20, 2023
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.

4 participants