Skip to content
This repository was archived by the owner on Oct 2, 2020. It is now read-only.

Reorganize code #6

Closed
wants to merge 1 commit into from
Closed

Reorganize code #6

wants to merge 1 commit into from

Conversation

abhinav
Copy link
Contributor

@abhinav abhinav commented May 24, 2017

This moves all executables to cmd/ and library code to the top-level.

Resolves #5


Note that this will break existing callers that do,

go get go.uber.org/tools/{update-license,parallel-exec}

To ensure that nobody breaks, we can,

  • Move their main() implementations into internal packages which
    export a Main() function. This function will be called from both,
    the new and old executable paths.
  • Alternatively, we can put placeholders there that just fail with a
    message indicating that the new path should be used.

Thoughts?

This moves all executables to cmd/ and library code to the top-level.

Resolves #5
@abhinav abhinav mentioned this pull request May 24, 2017
@abhinav
Copy link
Contributor Author

abhinav commented Dec 27, 2018

Stale. Abandoning.

@abhinav abhinav closed this Dec 27, 2018
@abhinav abhinav deleted the reorg branch December 27, 2018 14:38
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.

2 participants