Skip to content

[MOB-10290] create-loader-for-rniterableapi#612

Merged
lposen merged 4 commits into2.0.0-alpha/masterfrom
2.0.0-alpha/MOB-10290-create-loader-for-rniterableapi
Jan 6, 2025
Merged

[MOB-10290] create-loader-for-rniterableapi#612
lposen merged 4 commits into2.0.0-alpha/masterfrom
2.0.0-alpha/MOB-10290-create-loader-for-rniterableapi

Conversation

@lposen
Copy link
Contributor

@lposen lposen commented Nov 25, 2024

🔹 JIRA Ticket(s) if any

✏️ Description

Creates a loader for RNIterableAPI so that there is consistent error messaging when it is an issue.

@github-actions
Copy link

github-actions bot commented Nov 25, 2024

Lines Statements Branches Functions
Coverage: 38%
37.76% (176/466) 11.86% (21/177) 32.25% (50/155)

Copy link
Contributor

@evantk91 evantk91 left a comment

Choose a reason for hiding this comment

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

Looks great! nice reorganization.

// TODO: Create a loader for this
const RNIterableAPI = NativeModules.RNIterableAPI;

/**
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice this makes it much cleaner and maintainable

Base automatically changed from 2.0.0-alpha/MOB-10208-fix-npmignore to 2.0.0-alpha/master January 6, 2025 19:52
@lposen lposen merged commit ae80c18 into 2.0.0-alpha/master Jan 6, 2025
2 of 3 checks passed
@lposen lposen deleted the 2.0.0-alpha/MOB-10290-create-loader-for-rniterableapi branch January 6, 2025 19:53
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