Skip to content

Conversation

@hgiasac
Copy link
Collaborator

@hgiasac hgiasac commented Dec 30, 2025

This pull request improves the documentation and schema annotations for environment variable wrapper types across the codebase, making the intent and usage of these fields clearer for both developers and tools that consume the schema. Additionally, it disables the lll linter in the GolangCI configuration.

The most important changes are:

Schema annotation and documentation improvements:

  • Added descriptive description fields to the Value and Variable struct tags for all environment wrapper types (such as EnvAny, EnvString, EnvInt, EnvBool, EnvFloat, EnvMapString, EnvMapInt, EnvMapFloat, EnvMapBool, EnvStringSlice, EnvIntSlice, EnvFloatSlice, EnvBoolSlice). These descriptions clarify that Value is the default literal if the environment variable is empty, and Variable is the environment variable to be evaluated. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13]

Linting configuration:

  • Disabled the lll (line length linter) in .golangci.yml to allow longer lines, which are now required due to the more descriptive struct tags.

@hgiasac hgiasac merged commit f9737f9 into main Dec 30, 2025
1 check passed
@hgiasac hgiasac deleted the add-missing-jsonschema-comments branch December 30, 2025 16:00
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.

1 participant