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

fix: Change IntoApp::into_app to CommandFactory::command #3473

Merged
merged 2 commits into from Feb 15, 2022

Commits on Feb 14, 2022

  1. fix: Update derive->Command function name

    No good solution for transitioning the trate name, unfortnately, since
    we can't mark `use`s as deprecated (we can, it just does nothing).
    
    I got rid of the `into` prefix because that implies a `self` parameter
    that doesn't exist.
    epage committed Feb 14, 2022
    Copy the full SHA
    c3fec1f View commit details
    Browse the repository at this point in the history

Commits on Feb 15, 2022

  1. fix: Rename IntoApp to CommandFactory

    This is part of the `App` rename.
    
    Previously, I was concerned about not being able to deprecate
    
    For backwards compatibility, we still expose the `IntoApp` name.
    epage committed Feb 15, 2022
    Copy the full SHA
    ddac492 View commit details
    Browse the repository at this point in the history