WIP: minor improvements #397

						
				@ -6,4 +6,3 @@

					"coopcloud.tech/abra/pkg/config"

					"coopcloud.tech/abra/pkg/formatter"

					"coopcloud.tech/abra/pkg/recipe"

					recipePkg "coopcloud.tech/abra/pkg/recipe"

p4u1 commented

2024-01-06 15:57:03 +00:00

I think "coopcloud.tech/abra/pkg/recipe" should be the preferd way to import the package. Or is there a specific reason why you changed the import everywhere?

I think `"coopcloud.tech/abra/pkg/recipe"` should be the preferd way to import the package. Or is there a specific reason why you changed the import everywhere?

wykwit commented

2024-01-06 16:17:32 +00:00

I changed it wherever there was a double import, because in some places recipe can either be a variable or a package. It was used inconsistently within those source files, referring to the package sometimes as recipePkg and sometimes as just recipe which is additionally confusing when recipe is also used there as a common variable name. There are still some other places where the import is not renamed, but when the package name gets overshadowed it's not used anymore.

I changed it wherever there was a double import, because in some places `recipe` can either be a variable or a package. It was used inconsistently within those source files, referring to the package sometimes as `recipePkg` and sometimes as just `recipe` which is additionally confusing when `recipe` is also used there as a common variable name. There are still some other places where the import is not renamed, but when the package name gets overshadowed it's not used anymore.