Skip to content

Conversation

@PulokSaha0706
Copy link
Contributor

No description provided.

Signed-off-by: Pulok Saha <puloksaha@appscode.com>
@anisurrahman75
Copy link
Member

In the pkg/util.go create a method like below:

func removeDirWithLogErr(dir string) {
	if err := os.RemoveAll(dir); err != nil {
		klog.Errorf("failed to remove directory %s. Reason: %v", dir, err)
	}
}

And replace the os.RemoveAll(ScratchDir) call with removeDirWithLogErr(ScratchDir). If the prefix is defer, then keep the defer word also.

Signed-off-by: Arnab Baishnab Nipun <nipun@appscode.com>
Signed-off-by: Arnab Baishnab Nipun <nipun@appscode.com>
Signed-off-by: Arnab Baishnab Nipun <nipun@appscode.com>
anisurrahman75
anisurrahman75 previously approved these changes Dec 30, 2025
Signed-off-by: Arnab Baishnab Nipun <nipun@appscode.com>
@tamalsaha tamalsaha merged commit 25438da into master Dec 30, 2025
5 checks passed
@tamalsaha tamalsaha deleted the Update-gen branch December 30, 2025 14: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.

5 participants