Skip to content
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

MNG-6681 dependency type = extension+classifier #254

Merged
merged 1 commit into from
Jun 16, 2019
Merged

Conversation

hboutemy
Copy link
Member

removed misleading "While it usually represents the extension on the filename of the dependency, that is not always the case. A type can be mapped to a different extension and a classifier. The type often corresponds to the packaging used, though this is also not always the case."

@michael-o
Copy link
Member

michael-o commented Jun 16, 2019

Don't forget to mention that the classifier is optional. The title implies that it is not.

@hboutemy
Copy link
Member Author

good idea, I'll put "an optional classifier"

removed misleading "While it usually represents the extension on the
filename of the dependency, that is not always the case. A type can be
mapped to a different extension and a classifier. The type often
corresponds to the packaging used, though this is also not always the
case."
@hboutemy hboutemy merged commit 58de88f into master Jun 16, 2019
@hboutemy hboutemy deleted the MNG-6681 branch June 16, 2019 14:55
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.

3 participants