Skip to content

chore: up nitro, up iOS (6.12.0), android(10.5.0), keep nitrogen generated in git#2

Merged
mfazekas merged 3 commits intomainfrom
up-nitro-remove-nitrogen-from-code
Oct 29, 2025
Merged

chore: up nitro, up iOS (6.12.0), android(10.5.0), keep nitrogen generated in git#2
mfazekas merged 3 commits intomainfrom
up-nitro-remove-nitrogen-from-code

Conversation

@mfazekas
Copy link
Copy Markdown
Collaborator

@mfazekas mfazekas commented Oct 28, 2025

  • up nitro(0.31.3)
  • up iOS (6.12.0)
  • android(10.5.0) - required override in androidx.core:core,
  • keep nitrogen/generated in git

@mfazekas mfazekas requested a review from HayesGordon October 28, 2025 17:07
"eslint-plugin-prettier": "^5.2.3",
"jest": "^29.7.0",
"nitro-codegen": "^0.25.2",
"nitrogen": "^0.31.3",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nice, i remember seeing somewhere that this name might change

run: |
yarn nitrogen
if ! git diff --exit-code nitrogen/generated/; then
echo "Error: nitrogen/generated/ is out of date. Please run 'yarn nitrogen' and commit the changes."
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice

Copy link
Copy Markdown
Contributor

@HayesGordon HayesGordon left a comment

Choose a reason for hiding this comment

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

LGTM! Couple comments about whether we can use name or combine the dispose call for all resource cleanup

@mfazekas mfazekas merged commit a4e973b into main Oct 29, 2025
5 checks passed
mfazekas added a commit that referenced this pull request Nov 17, 2025
…rated in git (#2)

* chore: include nitro generated files in repo

* chore: upgrade ios and android runtime

* chore: upgrade nitro-modules
@HayesGordon HayesGordon deleted the up-nitro-remove-nitrogen-from-code branch December 9, 2025 16:25
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