Skip to content

feat: add comments to diff for React Native 0.74 - #384

Merged
cortinico merged 2 commits into
react-native-community:masterfrom
szymonrybczak:fix/74/add-proper-info-about-yarn-3
Apr 25, 2024
Merged

cortinico merged 2 commits into
react-native-community:masterfrom
szymonrybczak:fix/74/add-proper-info-about-yarn-3

Conversation

@szymonrybczak

Copy link
Copy Markdown
Contributor

Summary

For new project created with React Native Community CLI, we'll try to bump Yarn version to 3.6.4 which is the supported and maintained one. For people that are upgrading apps in my opinion it's very important to say what this diff means, and how they should handle it inside exciting projects.

Builds on top of: react-native-community/rn-diff-purge#75

Test Plan

Get diff from this PR react-native-community/rn-diff-purge#75 and check if the comment is shown properly in right place.

Checklist

  • I tested this thoroughly
  • I added the documentation in README.md (if needed)

@szymonrybczak

Copy link
Copy Markdown
Contributor Author

I'm wondering should also add this comment to line added inside package.json? 🤔

@lucasbento lucasbento left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice work, thank you for doing this!

@lucasbento

lucasbento commented Apr 25, 2024 •

Copy link
Copy Markdown
Member

I'm wondering should also add this comment to react-native-community/rn-diff-purge#75 added inside package.json? 🤔

Given package.json is at the top, sounds like a good idea!

@pvinis pvinis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

very nice! thank you

const release: ReleaseT = {
usefulContent: {
description:
'React Native 0.74 includes Yoga 3.0, Bridgeless by default under the New Architecture, batched onLayout updates, Yarn 3, removal of previously deprecated PropTypes, and some breaking changes, including updates to PushNotificationIOS. The Android Minimum SDK is now 23 (Android 6.0).',

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

💜

Comment thread src/releases/react-native/0.74.tsx Outdated
},
comments: [
{
fileName: '.yarnrc',

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

💜

@thymikee thymikee left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@szymonrybczak
szymonrybczak force-pushed the fix/74/add-proper-info-about-yarn-3 branch from 4114d5d to 6233994 Compare April 25, 2024 13:45
@cortinico
cortinico merged commit 06f61c3 into react-native-community:master Apr 25, 2024
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.

5 participants