Skip to content

Conversation

@ZgjimHaziri
Copy link
Contributor

Description

Implemented git integration based on this PoC and this spike doc.
Changes from the initial design:

  • Renamed back to git from vcs after the feedback we got and the research that git is mainly used by a large margin.
  • Marked the commands and options as beta.
  • Fixed minor issues
  • Changes in the push to branch functionality,
    • Fixed how it behaves when checking out on new branches that don't exist in the remote.
    • Now it cleanups the directory and pushes the new changes, in order to avoid having multiple directories (because of the version in the directory name) when we have exported new versions.

Bumped the minor version since it's a new feature being added.

Checklist

  • I have self-reviewed this PR
  • I have tested the change and proved that it works in different scenarios
  • I have updated docs if needed

@ZgjimHaziri ZgjimHaziri requested a review from a team as a code owner August 7, 2025 14:07
@ZgjimHaziri ZgjimHaziri mentioned this pull request Aug 7, 2025
3 tasks
Buqeta
Buqeta previously approved these changes Aug 8, 2025
promeris
promeris previously approved these changes Aug 8, 2025
jetakasabaqi
jetakasabaqi previously approved these changes Aug 8, 2025
@ZgjimHaziri ZgjimHaziri dismissed stale reviews from jetakasabaqi, promeris, and Buqeta via ff84e0a August 8, 2025 09:46
ksalihu
ksalihu previously approved these changes Aug 8, 2025
Buqeta
Buqeta previously approved these changes Aug 8, 2025
LaberionAjvazi
LaberionAjvazi previously approved these changes Aug 8, 2025
promeris
promeris previously approved these changes Aug 8, 2025
jetakasabaqi
jetakasabaqi previously approved these changes Aug 11, 2025
jetakasabaqi
jetakasabaqi previously approved these changes Aug 11, 2025
@ZgjimHaziri ZgjimHaziri merged commit 9caeb11 into master Aug 11, 2025
1 check passed
@ZgjimHaziri ZgjimHaziri deleted the TA-4066-implement-git-integration branch August 11, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

7 participants