Skip to content

Make default CI steps togglable - #275

Merged
armanbilge merged 1 commit into
typelevel:series/0.4from
armanbilge:issue/247
May 27, 2022
Merged

Make default CI steps togglable#275
armanbilge merged 1 commit into
typelevel:series/0.4from
armanbilge:issue/247

Conversation

@armanbilge

Copy link
Copy Markdown
Member

Closes #247.

This PR adds settings to the TypelevelCiPlugin to toggle various steps in the generated CI workflow.

This makes it easy for individual builds/projects to turn these steps on/off, so that a project can still get scalafmt checks for example even if they don't want header checks or scalac settings.

In sbt-typelevel v0.5.0 I will make all these false by default in the TypelevelCiPlugin, and enable them as appropriate in the TypelevelCiReleasePlugin and TypelevelPlugin. This will make it easier to use the TypelevelCiPlugin for non-publishing applications developed on GitHub.

@armanbilge armanbilge changed the title Make default CI steps configurable Make default CI steps togglable May 17, 2022

@DavidGregory084 DavidGregory084 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great to me! 💯

Comment on lines +35 to +36
lazy val tlCiScalafmtCheck =
settingKey[Boolean]("Whether to do scalafmt check in CI (default: false)")

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@danicheg does this address your usecase from #247? You will still have to add the scalafmt sbt plugin to your build, but you can enable the CI check with this setting.

@armanbilge

Copy link
Copy Markdown
Member Author

@DavidGregory084 feel free to merge this branch into #274 and keep going, I never force-push :)

@armanbilge
armanbilge merged commit 92a83ee into typelevel:series/0.4 May 27, 2022
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.

Proposal: move sbt-scalafmt to the TypelevelKernelPlugin

2 participants