-
Notifications
You must be signed in to change notification settings - Fork 144
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
move launch files into launch.legacy namespace #73
Conversation
c10c060
to
cf89281
Compare
cf89281
to
71d8ef9
Compare
This is ready for review. |
Would the history be preserved if the move would be done in two steps: one commit just moving the file, another commit modifying the files? If yes, I would advocate for that. |
No, it wouldn't appear that way on Github, but either way you do it, it is available via github's blame history and on the command line with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since CI passed this LGTM.
Btw, the feature was requested but never went anywhere, but there is a chrome extension to help with it: |
I'll take the positive review to apply to the other pr's too since they're so automatically derivative. Thanks! |
Was there a reason why the |
No reason that’s fine for me. |
In preparation for opening pull requests for the new launch API I've been working on, this pull request (and the connected ones) will simply move the existing launch system into a sub package (called
legacy
) and updates anything using launch to also use this new API.That way I can introduce the new API in smaller steps without first having to have it completely replace the existing tool and work in all cases.
This is the end goal, but there are a few things in
launch_testing
and a few of the output handlers I still need to port before I can start doing that. Plus this gives us something to compare and fallback to as we start using/tryout the new tool.