Skip to content

Commit

Permalink
fix: disallow files containing unwanted extension files when initiali…
Browse files Browse the repository at this point in the history
…zing a project
  • Loading branch information
afzal442 committed Jan 21, 2025
1 parent 9d93a04 commit bb1b899
Showing 1 changed file with 15 additions and 13 deletions.
28 changes: 15 additions & 13 deletions pkg/plugins/golang/v4/init.go
Original file line number Diff line number Diff line change
Expand Up @@ -187,21 +187,23 @@ func checkDir() error {
if isCapitalized && info.Name() != "PROJECT" {
return nil
}
// Allow files in the following list
allowedFiles := []string{
"go.mod", // user might run `go mod init` instead of providing the `--flag` at init
"go.sum", // auto-generated file related to go.mod
// Deny files with .go or .yaml extensions
disallowedExtensions := []string{
".go",
".yaml",
".mod",
".sum",
}
for _, allowedFile := range allowedFiles {
if info.Name() == allowedFile {
return nil
for _, ext := range disallowedExtensions {
if strings.HasSuffix(info.Name(), ext) {
return fmt.Errorf(

Check failure on line 199 in pkg/plugins/golang/v4/init.go

View workflow job for this annotation

GitHub Actions / golangci-lint

File is not properly formatted (gofmt)
"target directory is not empty and contains a disallowed file %q \".\", "+
"files with the following extensions [%s] are not allowed to avoid conflicts with the tooling.",
path, strings.Join(disallowedExtensions, ", "))
}
}
// Do not allow any other file
return fmt.Errorf(
"target directory is not empty (only %s, files and directories with the prefix \".\", "+
"files with the suffix \".md\" or capitalized files name are allowed); "+
"found existing file %q", strings.Join(allowedFiles, ", "), path)
}

Check failure on line 204 in pkg/plugins/golang/v4/init.go

View workflow job for this annotation

GitHub Actions / golangci-lint

File is not properly formatted (goimports)

return nil
})
if err != nil {
return err
Expand Down

0 comments on commit bb1b899

Please sign in to comment.