-
-
Notifications
You must be signed in to change notification settings - Fork 227
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
interactively query for license, suggest SPDX licenses #1902
Conversation
Thanks for your pull request, @WebFreak001! |
4a14ea6
to
0c582d9
Compare
updated, now allows entering raw values if they are present in the list (e.g. sdl, json; for backwards compatibility) and displays items in column order, not in row order now. (User feedback with sample size of 2 preferred this ordering) Example output:
|
322d0f9
to
f3a9598
Compare
lgtm i think |
just saw the license list is actually problematic, will adjust it to use what's documented on dub right now. We can change it later for properly specified licenses, e.g. #2403 |
@Geod24 what do you think of this one to start the transition to SPDX? Next I would change the dub docs to suggest using SPDX licenses and then inside DUB add an SPDX checker and emit warnings when the used license is invalid. |
- add select input (also for sdl/json) - provide common license defaults (choosealicense.com options + Boost) - order options by popularity
users liked it better if indexing went from up to down first, than from left to right.
d5dd81b
to
0e83529
Compare
✅ PR OK, no changes in deprecations or warnings Total deprecations: 14 Total warnings: 0 Build statistics: statistics (-before, +after)
-executable size=5329096 bin/dub
+executable size=5357768 bin/dub
rough build time=80s Full build output
|
This is a rebase of #1101