Skip to content

fix(types): fix several suppressed type errors in src/utils #7068

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

serhalp
Copy link
Member

@serhalp serhalp commented Feb 27, 2025

Summary

Requires netlify/build#6104 to pass

To do

Comment on lines -730 to +772
req.headers['x-forwarded-for'] = req.connection.remoteAddress || ''
req.headers['x-forwarded-for'] = req.socket.remoteAddress || ''
Copy link
Member Author

Choose a reason for hiding this comment

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

@serhalp serhalp force-pushed the types/fix-utils-ts branch 2 times, most recently from 4cce78d to b8c74ba Compare February 28, 2025 11:44
Copy link

github-actions bot commented Feb 28, 2025

📊 Benchmark results

Comparing with e3a7df7

  • Dependency count: 1,186 (no change)
  • Package size: 282 MB ⬇️ 0.00% decrease vs. e3a7df7
  • Number of ts-expect-error directives: 652 ⬇️ 11.81% decrease vs. e3a7df7

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant