Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

[BREAKING][cli] remove deprecated command functionality #4694

Closed
wants to merge 1 commit into from

Conversation

quinlanj
Copy link
Member

Why

Applying feedback from #4686 (review)

This PR removes deprecated command functionality. These changes should be made into a new major version release .

Test Plan

  • current test still passes

@quinlanj quinlanj requested a review from ide April 21, 2023 20:21
Copy link
Member

@ide ide left a comment

Choose a reason for hiding this comment

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

Related to this, we'll need to update the docs and remove references to these commands. For instance, this refers to expo push:android:upload. Would be good to grep the docs for any references to the removed commands.

Copy link

@huynhdev24 huynhdev24 left a comment

Choose a reason for hiding this comment

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

Instead of deleting can we show a warning deprecated?

@ide
Copy link
Member

ide commented Aug 4, 2023

@huynhdev24 When the server endpoints are removed, this command will stop working. The expo-cli package as a whole is deprecated, however, in favor of the versioned CLI invoked with npx expo.

@byCedric byCedric force-pushed the @quin/removeDeprecatedFn branch from 1063a0e to 1ca4012 Compare December 22, 2023 13:48
@byCedric byCedric force-pushed the @quin/removeDeprecatedFn branch from 1ca4012 to b8fab6d Compare December 22, 2023 15:57
@byCedric byCedric added expo:internal-discussion PRs that require Expo internal discussions and removed waiting labels Dec 23, 2023
@byCedric byCedric removed the expo:internal-discussion PRs that require Expo internal discussions label Jan 4, 2024
@byCedric
Copy link
Member

byCedric commented Jan 5, 2024

Closing this due to inactivity.

@byCedric byCedric closed this Jan 5, 2024
@ide
Copy link
Member

ide commented Jan 5, 2024

@quinlanj does it make sense to later revisit this, or abandon?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants