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

Add tests for export #59

Merged
merged 6 commits into from
Nov 8, 2024
Merged

Add tests for export #59

merged 6 commits into from
Nov 8, 2024

Conversation

ushitora-anqou
Copy link
Contributor

@ushitora-anqou ushitora-anqou commented Oct 31, 2024

The commits included in this PR should be rebased after #54 is merged into the main branch. done.

@ushitora-anqou ushitora-anqou force-pushed the add-tests-for-export branch 10 times, most recently from c361530 to 7c7184f Compare November 7, 2024 02:00
@ushitora-anqou ushitora-anqou marked this pull request as ready for review November 7, 2024 02:47
Copy link
Contributor

@satoru-takeuchi satoru-takeuchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ushitora-anqou I reviewed all commits besides the last one. I'll continue to do the remaining work while you reflect the existing comments.

Copy link
Contributor

@satoru-takeuchi satoru-takeuchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I completed to review this PR and left some additional minor suggestions.

internal/controller/mantlebackup_controller_test.go Outdated Show resolved Hide resolved
internal/controller/mantlebackup_controller_test.go Outdated Show resolved Hide resolved
internal/controller/mantlebackup_controller_test.go Outdated Show resolved Hide resolved
@ushitora-anqou
Copy link
Contributor Author

ushitora-anqou commented Nov 8, 2024

@satoru-takeuchi I fixed your points. Could you review this again?

Copy link
Contributor

@satoru-takeuchi satoru-takeuchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Please squash commits.

@ushitora-anqou
Copy link
Contributor Author

@satoru-takeuchi I squashed my commits. Could you review this PR again?

@satoru-takeuchi satoru-takeuchi merged commit 8b62289 into main Nov 8, 2024
3 checks passed
@satoru-takeuchi satoru-takeuchi deleted the add-tests-for-export branch November 8, 2024 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants