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

Clean up root directory to conform to standard go spec. #35

Merged
merged 1 commit into from
Dec 7, 2020

Conversation

issmirnov
Copy link
Owner

  • There's no need to make separate packages for the zap internals, as it
    would be just a single file + test per package.
  • We have to update the build commands to specify the cmd dir.

@issmirnov issmirnov force-pushed the refactor-codebase branch 2 times, most recently from 7cdee5f to 667cfbd Compare December 6, 2020 22:48
- There's no need to make separate packages for the zap internals, as it
  would be just a single file + test per package.
- We have to update the build commands to specify the cmd dir.
@issmirnov issmirnov merged commit 0ce9172 into master Dec 7, 2020
@issmirnov issmirnov deleted the refactor-codebase branch December 7, 2020 00:14
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.

1 participant