Skip to content

refactor: rename root.go files to descriptive names#147

Open
LaurenceJJones wants to merge 1 commit into
mainfrom
refactor/rename-root-files
Open

refactor: rename root.go files to descriptive names#147
LaurenceJJones wants to merge 1 commit into
mainfrom
refactor/rename-root-files

Conversation

@LaurenceJJones

Copy link
Copy Markdown
Contributor

Rename root.go files in pkg and internal directories to more descriptive names based on their content:

  • pkg/dataset/root.go → dataset.go (main DataSet implementation)
  • pkg/host/root.go → manager.go (Host Manager implementation)
  • pkg/spoa/root.go → spoa.go (SPOA handler implementation)
  • internal/appsec/root.go → appsec.go (AppSec implementation)
  • internal/geo/root.go → database.go (GeoDatabase implementation)
  • internal/remediation/root.go → remediation.go (Remediation types)
  • internal/remediation/ban/root.go → ban.go (Ban implementation)
  • internal/remediation/captcha/root.go → captcha.go (Captcha implementation)

This improves code discoverability by using names that reflect the actual functionality instead of the generic "root.go" convention.

to make @blotus happier :D

Rename root.go files in pkg and internal directories to more
descriptive names based on their content:

- pkg/dataset/root.go → dataset.go (main DataSet implementation)
- pkg/host/root.go → manager.go (Host Manager implementation)
- pkg/spoa/root.go → spoa.go (SPOA handler implementation)
- internal/appsec/root.go → appsec.go (AppSec implementation)
- internal/geo/root.go → database.go (GeoDatabase implementation)
- internal/remediation/root.go → remediation.go (Remediation types)
- internal/remediation/ban/root.go → ban.go (Ban implementation)
- internal/remediation/captcha/root.go → captcha.go (Captcha implementation)

This improves code discoverability by using names that reflect the
actual functionality instead of the generic "root.go" convention.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

@sabban sabban left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

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