Skip to content

swift: add DrString tool & fix documentation comment issues#2233

Merged
jpsim merged 3 commits intomainfrom
swift-add-drstring-tool--fix-documentation-comment-issues
May 5, 2022
Merged

swift: add DrString tool & fix documentation comment issues#2233
jpsim merged 3 commits intomainfrom
swift-add-drstring-tool--fix-documentation-comment-issues

Conversation

@jpsim
Copy link
Contributor

@jpsim jpsim commented May 2, 2022

This will help ensure that Swift docstrings are in the correct format and that all documented interfaces are complete.

jpsim added 2 commits May 2, 2022 10:20
Signed-off-by: JP Simard <jp@jpsim.com>
Signed-off-by: JP Simard <jp@jpsim.com>
@jpsim jpsim marked this pull request as ready for review May 2, 2022 14:27
@jpsim jpsim requested review from Augustyniak and keith May 2, 2022 14:27
@jpsim
Copy link
Contributor Author

jpsim commented May 2, 2022

/retest

Copy link
Contributor

@Augustyniak Augustyniak left a comment

Choose a reason for hiding this comment

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

This is great 💯

…ol--fix-documentation-comment-issues

* origin/main:
  Fix `bump_lyft_support_rotation.sh` posting to Slack (#2234)
  jni: fix mismatched return value types (#2252)
  Remove bintray as a maven source (#2248)
  test: refine connection drain test (#2245)
  cleanup: remove comment in Http::Client that no longer applies (#2203)
  cleanup: fix test with inaccurate description (#2202)
  network: perform post-DNS connection drain on a per-host basis (#2240)
  network: add enableDrainPostDnsRefresh to iOS (#2242)
  envoy: update to em-cherry (#2241)
  network: support draining connections after triggered DNS refresh (#2225)
  Bump rules_apple to 0.34.2 (#2236)
  Bump Lyft Support Rotation (#2232)

Signed-off-by: JP Simard <jp@jpsim.com>
@jpsim jpsim merged commit 2c56c0f into main May 5, 2022
@jpsim jpsim deleted the swift-add-drstring-tool--fix-documentation-comment-issues branch May 5, 2022 22:22
jpsim added a commit that referenced this pull request May 6, 2022
…on-6.0.0-pre.20220421.3

* origin/main:
  Upgrade rules_jvm_external from 4.1 -> 4.2 (#2254)
  CancelProofEnvoyStream (#2250)
  swift: add DrString tool & fix documentation comment issues (#2233)
  Fix `bump_lyft_support_rotation.sh` posting to Slack (#2234)
  jni: fix mismatched return value types (#2252)
  Remove bintray as a maven source (#2248)
  test: refine connection drain test (#2245)
  cleanup: remove comment in Http::Client that no longer applies (#2203)
  cleanup: fix test with inaccurate description (#2202)
  network: perform post-DNS connection drain on a per-host basis (#2240)

Signed-off-by: JP Simard <jp@jpsim.com>
jpsim added a commit that referenced this pull request May 6, 2022
* main:
  envoy: update to efbbb04 (#2258)
  Update rules_java, rules_detekt, rules_jvm_external (#2249)
  Upgrade rules_jvm_external from 4.1 -> 4.2 (#2254)
  CancelProofEnvoyStream (#2250)
  swift: add DrString tool & fix documentation comment issues (#2233)
  Fix `bump_lyft_support_rotation.sh` posting to Slack (#2234)
  jni: fix mismatched return value types (#2252)
  Remove bintray as a maven source (#2248)
  test: refine connection drain test (#2245)
  cleanup: remove comment in Http::Client that no longer applies (#2203)
  cleanup: fix test with inaccurate description (#2202)
  network: perform post-DNS connection drain on a per-host basis (#2240)
  network: add enableDrainPostDnsRefresh to iOS (#2242)
  envoy: update to em-cherry (#2241)
  network: support draining connections after triggered DNS refresh (#2225)

Signed-off-by: JP Simard <jp@jpsim.com>
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