Skip to content

Conversation

@pyramation
Copy link
Contributor

@pyramation pyramation commented Dec 26, 2025

Summary

Makes MIT the default license option in create-gen-app by:

  1. Adding a DEFAULT_LICENSE = 'MIT' constant
  2. Adding a CLOSED_LICENSE = 'CLOSED' constant
  3. Modifying listSupportedLicenses() to return licenses in a specific order:
    • MIT first (as the default)
    • Other licenses alphabetically in the middle
    • CLOSED last

When boilerplate templates use "optionsFrom": "licenses", MIT will now appear as the first option and CLOSED will appear last.

Review & Testing Checklist for Human

  • Verify that having MIT first in the options list actually makes it the default selection in the prompts (test by running a scaffolding command that uses the license question)
  • Verify CLOSED appears last in the license list when prompted
  • Confirm the DEFAULT_LICENSE and CLOSED_LICENSE exports are useful for any downstream code that might need to reference these values

Notes

  • All existing tests pass (28 tests)
  • The sort order is: MIT → alphabetical (APACHE-2.0, BSD-3-CLAUSE, GPL-3.0, ISC, MPL-2.0, UNLICENSE) → CLOSED

Link to Devin run: https://app.devin.ai/sessions/a51e296fcb614c0294cf1678407a08f3
Requested by: Dan Lynch (pyramation@gmail.com) / @pyramation

- Add DEFAULT_LICENSE constant set to 'MIT'
- Modify listSupportedLicenses() to return MIT first in the list
- Other licenses remain sorted alphabetically after MIT
@devin-ai-integration
Copy link

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

- Add CLOSED_LICENSE constant
- Update sort to put CLOSED at the end of the list
- Order is now: MIT first, alphabetical in middle, CLOSED last
@pyramation pyramation merged commit fe11e2e into main Dec 26, 2025
34 checks passed
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