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

Enhancements to improve maintainability #145

Merged
merged 2 commits into from
Nov 8, 2024
Merged

Enhancements to improve maintainability #145

merged 2 commits into from
Nov 8, 2024

Conversation

morphy2k
Copy link
Owner

@morphy2k morphy2k commented Oct 29, 2024

This PR introduces several changes to simplify the code base, improve flexibility, and maintainability.

Key improvements

  • Optimized input handling and process execution: By moving these operations from Bash to the binary, we improve maintainability and make it possible to reduce the container image further.
  • Leveraged official Revive container image: By using the official Revive container image as the base image, we improve maintainability and simplify the build process.
  • Enabled cross-compilation: By using Go's cross-compilation capabilities, the compilation times are now much faster.
  • Implemented tests: The added tests ensure the correctness and robustness.

@morphy2k morphy2k added enhancement New feature or request docker Pull requests that update Docker code labels Oct 29, 2024
@morphy2k morphy2k changed the title Enhancements to improve maintainability Enhancements to improve maintainability & more Oct 29, 2024
@morphy2k morphy2k marked this pull request as ready for review November 8, 2024 21:30
@morphy2k morphy2k changed the title Enhancements to improve maintainability & more Enhancements to improve maintainability Nov 8, 2024
@morphy2k morphy2k merged commit 594f694 into master Nov 8, 2024
1 check passed
@morphy2k morphy2k deleted the refactor branch November 9, 2024 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker Pull requests that update Docker code enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant