-
Notifications
You must be signed in to change notification settings - Fork 64
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
Removes Sbt-org-policies Dependency #465
Conversation
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.
Looks good overall, however we're missing a few apache headers 👍
Also, I'm thinking there might be an existing library to perform all this file I/O? |
Co-Authored-By: Ben Fradet <benjamin.fradet@gmail.com>
…icrosites into removes-sbt-org-policies
Yeah, that would make sense, and I'm sure it exists something. However, I didn't want to make this PR too complicated needed to migrate the existing code in sbt-microsites to the potential new library. |
yes definitely, it was more a general remark than one specific to this PR 👍 |
The Github actions workflows are now providing what the sbt plugin was doing until now.
It also:
plugins.sbt
level. We have now someSBT
duplicated code, but hopefully, @scala-steward will take care of the dependencies.Exceptions
,Config.scala
,GitHubOps.scala
,FileReader.scala
,FileWriter.scala
,IOUtils.scala
, andYamlFormats.scala
.GitHubOps
algebra, apart from copying-paste, I've simplified it removing all theEitherT
types, apart from upgrading to the latest version of github4s. Original version 🔗 .Note: the microsite step will fail once this is merged in master, but that's expected (the solution is still in progress).