-
Notifications
You must be signed in to change notification settings - Fork 312
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
Read once git status and set repository SHA once for okteto up #3761
Comments
Can take this up if no one else is already working on it. |
@AnshulAngaria thanks for offering, this is a bit tricky because it requires an initial investigation to understand what approach to use and potentially do a refactor. It's also part of our critical path so we think it's important own this change at this time, thanks again 🙏 |
This issue is stale because it has been open for 60 days with no activity. Comment on this issue or it will be closed in 7 days |
Reopening this one since needs to be planned |
Describe the bug
Currently the CLI will repeatedly calculate the git status of the local repo in order to run certain commands. We've observed that this operation could sometime taking longer than expected, especially if the host doesn't have
git
installed. Therefore a refactor to this part is required to optimize the stability and performance of the CLI.Expected behavior
The git status should be calculated only once.
Additional context
sync.Once
The text was updated successfully, but these errors were encountered: