-
Notifications
You must be signed in to change notification settings - Fork 88
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
Separate benchmark and examples #52
Comments
Hello @aman00323
The repository structure is ancient. It was created in the era of first releases of
The reason I set up examples running on CI was an extra assurance that software works. We can add missing bits to unit tests (if needed) and skip running examples. Though I like having ready to use code that a developer can execute right after cloning the repo.
I'd love to move from Travis to GitHub actions Action Plan
Please feel free to take care from now. But please move forward with small PRs that fixes/improves one thing at a time. |
Sure @satyrius I will start with brushing the existing structure, then we will pick GitHub action. Also, I would like to suggest adding a few GitHub bots or actions that our life makes easier. https://github.com/marketplace/pull-request-size |
@satyrius I would like to discuss one more thing, How we see this package in the future.
based on this decision it would make it easy to separate two things
|
I'm would like to have everything in one place. It should not affect the build for any dependable project anyway. Please, provide me example of famous golang libraries that have examples and Dockerfile and what not, so we can use it a a guide. |
Dockerfile context is here if we want to serve user ready-to-use library just like the example. here is one example that I am aware of right now. |
Alright! Looking forward to seeing pull requests from you. |
@satyrius I have one more suggestion, what if we add auto-release with GitHub action? So when we merge into master it would make a new tag so we don't need to release |
We can make it after we finished with the refactoring. We'll release a new major version and will add some "version bump" action |
@satyrius would you add me as collaborator, so i can avoid pushing from my repo everytime. |
@aman00323 sure thing! I sent you the invite |
thanks, @satyrius |
Hey @satyrius,
Let me know your thoughts on this!
I would like it to take it further
The text was updated successfully, but these errors were encountered: