-
Notifications
You must be signed in to change notification settings - Fork 308
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Think about a better name for the RepositoryConfiguration
class
#4466
Labels
Comments
ProjectConfiguration |
OrtResultConfiguration |
DistributionConfiguration |
ModuleConfiguration |
sschuberth
added
enhancement
Issues that are considered to be enhancements
model
About the data model
labels
Sep 16, 2021
sschuberth
changed the title
Think about better names for the
Think about a better name for the Sep 16, 2021
RepositoryConfiguration
classRepositoryConfiguration
class
ProductConfiguration |
ArtifactConfiguration |
ReleaseArtifactConfiguration |
Closed as part of backlog grooming. Feel free to comment if you would like to contribute to this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
While the class is named RepositoryConfiguration, it is actually documented to apply to projects:
ort/model/src/main/kotlin/config/RepositoryConfiguration.kt
Lines 26 to 30 in 7bb799b
In general, the class should probably named after the entity the configuration applies to, not after where it is stored.
Please add some proposals for a better name to this issue. To start with, here are some ideas from the ORT developer meeting:
(See below, in separate comments each, so people can vote via reaction on the comment.)
The text was updated successfully, but these errors were encountered: