Skip to content

KRA-738 - Improvements #60

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.

Sign up for GitHub

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

Jump to bottom
Merged
matteogastaldello merged 3 commits into from
Aug 4, 2025
Merged

KRA-738 - Improvements #60

matteogastaldello merged 3 commits into from
Aug 4, 2025

Conversation

@matteogastaldello
Copy link
Collaborator

@matteogastaldello matteogastaldello commented Aug 4, 2025 •
edited
Loading

  • Support for override
  • non-existent folder on target repo
  • different path for .krateoignore -> added an additional field .spec.fromRepo.krateoIgnorePath (defaults to "/")
  • use disk instead of memory for repo FS

@matteogastaldello
feat: support for override, non-existent folder on target repo, diffe…
a9322dc 
…rent path for .krateoignore, use disk instead of memory for repo fs
@matteogastaldello matteogastaldello linked an issue Aug 4, 2025 that may be closed by this pull request
Revision #59
Closed
@matteogastaldello matteogastaldello self-assigned this Aug 4, 2025
@codecov
Copy link

codecov bot commented Aug 4, 2025

Codecov Report

❌ Patch coverage is 59.09091% with 27 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/clients/git/git.go 58.06% 20 Missing and 6 partials ⚠️
internal/controllers/repo/utils.go 75.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@matteogastaldello matteogastaldello merged commit abb598d into main Aug 4, 2025
4 checks passed
@matteogastaldello matteogastaldello changed the title Improvements KRA-738 - Improvements Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@matteogastaldello matteogastaldello

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Revision

2 participants

@matteogastaldello