Skip to content

Conversation

@isekovanic
Copy link
Contributor

🎯 Goal

This PR resolves a couple of housekeeping tasks:

  • Upgrade eslint and prettier to latest
    • This includes a migration to the new eslint.config.mjs format as well
    • It also includes reconciliation of rules that either got changed, removed or deprecated
    • It utilizes the new typescript-eslint configuration and all of its plugins and rules
    • An update of all eslint plugins as well
    • An update of styling in our SDK to reflect the new improvements better
    • Fixes in various places which were caught with the new rules (potentially even some very slight performance issues)
  • An upgrade of the node runner for our SDK to 22.x
    • This includes an update of both variants of the SDK
    • It includes an update of various node related actions
    • It also includes an update of all sample apps

It is a follow-up of this change.

🛠 Implementation details

🎨 UI Changes

iOS
Before After
Android
Before After

🧪 Testing

☑️ Checklist

  • I have signed the Stream CLA (required)
  • PR targets the develop branch
  • Documentation is updated
  • New code is tested in main example apps, including all possible scenarios
    • SampleApp iOS and Android
    • Expo iOS and Android

@isekovanic isekovanic changed the base branch from chore/rn-0.77-upgrade to develop February 23, 2025 14:18
@Stream-SDK-Bot
Copy link
Contributor

Stream-SDK-Bot commented Feb 23, 2025

SDK Size

title develop branch diff status
js_bundle_size 468 KB 468 KB 0 B 🟢

# Conflicts:
#	examples/ExpoMessaging/yarn.lock
#	examples/SampleApp/package.json
#	examples/SampleApp/src/hooks/useChatClient.ts
#	examples/TypeScriptMessaging/.node-version
@isekovanic isekovanic merged commit 057e5d2 into develop Feb 24, 2025
5 checks passed
@isekovanic isekovanic deleted the chore/eslint-sdk-upgrade branch February 24, 2025 12:35
@stream-ci-bot
Copy link
Contributor

🎉 This PR is included in version 6.6.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants