Skip to content

chore: [IOPLT-000] Remove unused postinstall scripts - #8450

Open
LazyAfternoons wants to merge 2 commits into
masterfrom
remove-unused-postinstall-scripts
Open

chore: [IOPLT-000] Remove unused postinstall scripts#8450
LazyAfternoons wants to merge 2 commits into
masterfrom
remove-unused-postinstall-scripts

Conversation

@LazyAfternoons

@LazyAfternoons LazyAfternoons commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Short description

This PR removes two unused iOS scripts.

List of changes proposed in this pull request

  • Remove add-ios-source-maps.sh as the code introduced [#149390457] Add support for source maps in Error Analytics #33 which was consuming it by is long gone and it currently fails during pnpm i:
    Custom 'postinstall' script: 'bin/add-ios-source-maps.sh' did not run.
  • Remove add-ios-env-config.sh as newer versions of react-native-config don't require it.

How to test

iOS build should work as before. Error reporting on Mixpanel should work as before.

@github-actions

Copy link
Copy Markdown
Contributor

Jira Pull Request Link

This Pull Request refers to Jira issues:

@github-actions

Copy link
Copy Markdown
Contributor

PR Title Validation for conventional commit type

All good! PR title follows the conventional commit type.

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.79%. Comparing base (9e83e7a) to head (242e42d).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8450   +/-   ##
=======================================
  Coverage   63.79%   63.79%           
=======================================
  Files        1978     1978           
  Lines       37598    37598           
  Branches     6360     6360           
=======================================
+ Hits        23985    23986    +1     
  Misses      12235    12235           
+ Partials     1378     1377    -1     

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ca3373f...242e42d. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dmnplb dmnplb added the 💀 Dead matter Graphic assets or chunks of code that are no longer referenced or needed label Aug 12, 2026
@LazyAfternoons
LazyAfternoons marked this pull request as ready for review August 24, 2026 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💀 Dead matter Graphic assets or chunks of code that are no longer referenced or needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants