Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[WTF-1735]: Fix PWT resolution/override versions #73

Conversation

legio-vi-ferrata
Copy link
Contributor

@legio-vi-ferrata legio-vi-ferrata commented Jan 8, 2024

Fix the misaligned versions for react, react-dom, react-native packages and their corresponding @types packages during migration.

Checklist

  • Contains unit tests ❌
  • Contains breaking changes ❌
  • Compatible with:
  • Did you update version and changelog? ❌
  • PR title properly formatted ([XX-000]: description)? ✅

This PR contains

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Other (describe)

What is the purpose of this PR?

The migration script creates resolutions and overrides for three packages in an incorrect way. This PR fixes the issue by aligning the versions of those packages in the migration script.

Relevant changes

Some hard coded values are adjusted and extracted into constants to be the same everywhere they are relavant.

What should be covered while testing?

An example widget should be upgraded using the standard procedure. Another PR in docs repository is on the way to clearly state this standard procedure of upgrade.

@legio-vi-ferrata legio-vi-ferrata force-pushed the wtf-1735-fix-override-and-resolution-versions-in-migration-script branch from 8f5fe38 to dab2102 Compare January 10, 2024 16:05
@legio-vi-ferrata legio-vi-ferrata force-pushed the wtf-1735-fix-override-and-resolution-versions-in-migration-script branch from ee17948 to aec3d3b Compare January 12, 2024 16:47
@legio-vi-ferrata legio-vi-ferrata merged commit 240b5e3 into master Jan 16, 2024
9 checks passed
@legio-vi-ferrata legio-vi-ferrata deleted the wtf-1735-fix-override-and-resolution-versions-in-migration-script branch January 16, 2024 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants