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

feat: add alias command as an eventual replacement for the 3 existing alias commands #572

Merged
merged 3 commits into from
Aug 4, 2024

Conversation

digitalsadhu
Copy link
Member

This is a non breaking change but does add deprecation messages for pkg-alias, map-alias and npm-alias, all of which are replaced by this new alias command.

The alias command detects type from eik.json which was not possible in earlier versions of the cli which was the reason for the existence of 3 separate commands instead of 1. If the command is run in a directory that does not contain an eik.json file, type can be provided via the command line argument --type. If neither are present, an error is thrown.

There is some formatting of command messages thrown in for good measure with this PR.

… alias commands

This is a non breaking change but does add deprecation messages for pkg-alias, map-alias and npm-alias, all of which
are replaced by this new alias command.

The alias command detects type from eik.json which was not possible in earlier versions of the cli which was the reason for the existence of 3 separate commands instead of 1.
If the command is run in a directory that does not contain an eik.json file, type can be provided via the command line argument --type. If neither are present, an error is thrown.
Copy link
Contributor

@leftieFriele leftieFriele left a comment

Choose a reason for hiding this comment

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

🌮 huge win doing this. much easier to grasp what's going on with Alias commands with just this one.

Copy link
Contributor

@wkillerud wkillerud left a comment

Choose a reason for hiding this comment

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

🧡

commands/alias.js Outdated Show resolved Hide resolved
commands/alias.js Outdated Show resolved Hide resolved
commands/alias.js Outdated Show resolved Hide resolved
commands/alias.js Outdated Show resolved Hide resolved
@digitalsadhu digitalsadhu merged commit bb0c381 into next Aug 4, 2024
2 checks passed
@digitalsadhu digitalsadhu deleted the add_alias_command branch August 4, 2024 20:33
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