search

Melkeydev / go-blueprint

Go-blueprint allows users to spin up a quick Go project using a popular framework
https://docs.go-blueprint.dev/
MIT License
5.88k stars 333 forks source link

[Feature Request] Clear Cache for golangci-lint #216

Closed H0llyW00dzZ closed 7 months ago

H0llyW00dzZ commented 7 months ago

Tell us about your feature request

image

[!NOTE] It's better to remove the cache automatically after running the jobs, because we don't really need it as Go is fast, unlike Rust or other compiled languages that have slow compilers.

Disclaimer

Ujstor commented 7 months ago

I need to check how to disable cache. Good idea!

H0llyW00dzZ commented 7 months ago

I need to check how to disable cache. Good idea!

@Ujstor I think it must be independently by creating another workflows for removing it

Ujstor commented 7 months ago

After taking a quick look at the cache issue, I'm not sure if I want to mess with it. The new CI and linter are good improved. I think expected benefits are not justifying the struggle.

https://stackoverflow.com/questions/63521430/clear-cache-in-github-actions

H0llyW00dzZ commented 7 months ago

After taking a quick look at the cache issue, I'm not sure if I want to mess with it. The new CI and linter are good improved. I think expected benefits are not justifying the struggle.

https://stackoverflow.com/questions/63521430/clear-cache-in-github-actions

That's it, I think, for removing the cache.

Because there are so many cache that we don't really needed, when looking at this: image

Ujstor commented 7 months ago

I thought cache is automatically deleted when it hits a limit... If not, then we need to create a workflow to clean it

H0llyW00dzZ commented 7 months ago

I thought cache is automatically deleted when it hits a limit... If not, then we need to create a workflow to clean it

@Ujstor I don't think so it will automatically deleted, It's better create a workflow to clean it

Ujstor commented 7 months ago

I am closing this for now. The cache is cleared when the PR is merged or closed. I will create an action to clean it if this becomes an issue in the future. 10GB is sufficient for now.