Skip to content

Conversation

@joe-yeager
Copy link
Contributor

@joe-yeager joe-yeager commented Apr 22, 2025

Description and Context

This pull request does the following:

  • Changes the default platform-version for hs app migrate to 2025.2
  • Adds a message when hs project migrate is ran outside of a project to nudge them to use hs app migrate if they are trying to migrate a legacy app to projects.
    Screenshot 2025-04-23 at 12 20 20 PM
  • Changes verbiage in log messages
  • Unhides apps associated with projects when hs app migrate is ran, and shows a message that they are only migratable with hs project migrate
    Screenshot 2025-04-23 at 12 22 32 PM
  • Fixes a bug where the default answer was an empty string when prompting for UIDs
  • Removes unstable as an option in the platform-version flag
  • Adds a hidden --unstable flag
  • Adds links to build and projects when the build is successful
  • Changes default proceed with migration prompt to No

hidden: true,
default: v2025_2,
},
unstable: {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The idea is that this will only be used for internal testing and shouldn't be used by external users. We will keep this hidden and keep unstable out of the list of platform version choices.

@joe-yeager joe-yeager marked this pull request as ready for review April 24, 2025 16:38
Copy link
Contributor

@kemmerle kemmerle left a comment

Choose a reason for hiding this comment

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

Changes make sense to me, and I like the updated copy.

@joe-yeager joe-yeager merged commit e57419d into main Apr 24, 2025
1 check passed
@joe-yeager joe-yeager deleted the jy/migrations-polish branch April 24, 2025 18:27
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.

3 participants