Skip to content

[ui][ios] - rename RNHostProtocol to RNHostViewProtocol#41519

Merged
intergalacticspacehighway merged 2 commits intomainfrom
rn-host-protocol-rename
Dec 9, 2025
Merged

[ui][ios] - rename RNHostProtocol to RNHostViewProtocol#41519
intergalacticspacehighway merged 2 commits intomainfrom
rn-host-protocol-rename

Conversation

@intergalacticspacehighway
Copy link
Copy Markdown
Contributor

Why

Missed this renaming in #41509

How

Test Plan

Checklist

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 9, 2025

Subscribed to pull request

File Patterns Mentions
packages/expo-modules-core/** @Kudo, @lukmccall
packages/expo-ui/** @aleqsio, @behenate, @douglowder

Generated by CodeMention

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Dec 9, 2025
Comment thread packages/expo-modules-core/ios/Core/Views/SwiftUI/SwiftUIViewHost.swift Outdated
Co-authored-by: Kudo Chien <kudo@expo.dev>
@expo-bot
Copy link
Copy Markdown
Collaborator

expo-bot commented Dec 9, 2025

Hi there! 👋 I'm a bot whose goal is to ensure your contributions meet our guidelines.

I've found some issues in your pull request that should be addressed (click on them for more details) 👇

⚠️ Suggestion: Missing changelog entries


Your changes should be noted in the changelog, e.g.:
- [ios] - rename RNHostProtocol to RNHostViewProtocol ([#41519](https://github.com/expo/expo/pull/41519) by [@intergalacticspacehighway](https://github.com/intergalacticspacehighway))
Read Updating Changelogs guide and consider adding an appropriate entry to the following changelogs:


Generated by ExpoBot 🤖 against ef80823

@expo-bot
Copy link
Copy Markdown
Collaborator

expo-bot commented Dec 9, 2025

The Pull Request introduced fingerprint changes against the base commit: a103483

Fingerprint diff
[
  {
    "op": "changed",
    "beforeSource": {
      "type": "dir",
      "filePath": "../../packages/expo-modules-core",
      "reasons": [
        "expoAutolinkingIos",
        "expoAutolinkingAndroid",
        "expoAutolinkingIos"
      ],
      "hash": "a5a69d3e95cca28112bddd2e497dbf29bcf08a01"
    },
    "afterSource": {
      "type": "dir",
      "filePath": "../../packages/expo-modules-core",
      "reasons": [
        "expoAutolinkingIos",
        "expoAutolinkingAndroid",
        "expoAutolinkingIos"
      ],
      "hash": "e2a72efbe83f07203ad8fea8169e7c5a5071c412"
    }
  },
  {
    "op": "changed",
    "beforeSource": {
      "type": "dir",
      "filePath": "../../packages/expo-ui/ios",
      "reasons": [
        "expoAutolinkingIos"
      ],
      "hash": "7c2ef04c27856dd6fc92c6b3411f3d7cdc4d7e55"
    },
    "afterSource": {
      "type": "dir",
      "filePath": "../../packages/expo-ui/ios",
      "reasons": [
        "expoAutolinkingIos"
      ],
      "hash": "c945f781ee51d418b9abfb15e9e71fd650a294b3"
    }
  }
]

Generated by PR labeler 🤖

@intergalacticspacehighway intergalacticspacehighway merged commit 80ffc01 into main Dec 9, 2025
17 of 20 checks passed
@intergalacticspacehighway intergalacticspacehighway deleted the rn-host-protocol-rename branch December 9, 2025 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants