-
Notifications
You must be signed in to change notification settings - Fork 737
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
Release 6.0.0 #509
Release 6.0.0 #509
Conversation
I'll do a PR to link to the new wiki page about swiftlint custom rules before we merge this PR |
(We should also include #508 in this release) |
4f1315b
to
cf4f408
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be the best occasion to update our internal release process doc in our wiki (which is supposed to serve as a guide to ourselves, really): https://github.com/SwiftGen/SwiftGen/wiki/%5BAdmin%5D-Release-Process
I've updated the |
@AliSoftware right, so I wrongly assumed that the CLI template name option was called We can either leave them as is, or add new options that match those docs and deprecate the old one (see last commit 05e0b88). Not sure if that warrants a mention in the MG, as the old option is still around. |
Wait seriously? It was still |
So iiuc basically what you did in the last commit, right? |
Jep. |
Question is: changelog entry and MG entry? Or skip 'em? |
CHANGELOG entry I think yes, we need to trace that somewhere at least. MG entry probably not worth it, the overall suggestion in the MG already being basically "Y U Not use a config file already?" anyway 😜 |
TODO: Update the playground + |
…hanges" is at the top
8605b6b
to
538a211
Compare
I've moved the "bug fixes" changelog section to be after "new features". I've adapted the rake script as well. The order is now: