Skip to content

fix: set codegen config type property to 'modules' instead of 'all'#221

Merged
chmjkb merged 1 commit intov0.4.0-rc1from
@chmjkb/codegen-config-fix
May 19, 2025
Merged

fix: set codegen config type property to 'modules' instead of 'all'#221
chmjkb merged 1 commit intov0.4.0-rc1from
@chmjkb/codegen-config-fix

Conversation

@chmjkb
Copy link
Copy Markdown
Collaborator

@chmjkb chmjkb commented Apr 25, 2025

Description

This PR changes the configuration of the config to 'modules' instead of 'all', since we don't generate code for native Fabric components.
See https://reactnative.dev/docs/the-new-architecture/using-codegen#configuring-codegen

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (improves or adds clarity to existing documentation)

Tested on

  • iOS
  • Android

Testing instructions

Screenshots

Related issues

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@chmjkb chmjkb changed the title fix: set codegen config to 'modules'instead of 'all' fix: set codegen config to 'modules' instead of 'all' Apr 25, 2025
@chmjkb chmjkb changed the title fix: set codegen config to 'modules' instead of 'all' fix: set codegen config type property to 'modules' instead of 'all' Apr 25, 2025
@chmjkb chmjkb changed the base branch from main to v0.4.0-rc1 May 19, 2025 06:14
@chmjkb chmjkb force-pushed the @chmjkb/codegen-config-fix branch from 9c49f75 to 2bfc446 Compare May 19, 2025 06:18
@chmjkb chmjkb merged commit 2f38823 into v0.4.0-rc1 May 19, 2025
@chmjkb chmjkb deleted the @chmjkb/codegen-config-fix branch May 19, 2025 06:19
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