Skip to content

Setup sbt-typelevel#1968

Merged
zarthross merged 15 commits intocirce:mainfrom
armanbilge:topic/sbt-typelevel
Jul 11, 2022
Merged

Setup sbt-typelevel#1968
zarthross merged 15 commits intocirce:mainfrom
armanbilge:topic/sbt-typelevel

Conversation

@armanbilge
Copy link
Contributor

@armanbilge armanbilge commented Jun 2, 2022

As discussed in #1958 (comment).

Currently the website build is broken for unclear reasons (besides it's behind a major version in mdoc). I'm loathe to sink time into it if it's going to be replaced by the sbt-typelevel-site plugin. Edit: maybe I fixed it ...

@armanbilge armanbilge requested a review from zarthross as a code owner June 2, 2022 16:17

def Scala3 = "3.1.0"
ThisBuild / tlBaseVersion := "0.14"
ThisBuild / tlCiReleaseTags := false
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This disables CI release, but say the word (and configure the secrets) and I'll switch it on :)

@codecov-commenter
Copy link

codecov-commenter commented Jun 2, 2022

Codecov Report

Merging #1968 (21fb4f3) into main (12bfdbf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1968   +/-   ##
=======================================
  Coverage   86.02%   86.02%           
=======================================
  Files          74       74           
  Lines        2554     2554           
  Branches      136      120   -16     
=======================================
  Hits         2197     2197           
  Misses        357      357           
Flag Coverage Δ
2.12.15 85.81% <ø> (-0.12%) ⬇️
2.13.8 86.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../main/scala-2/io/circe/benchmark/AtBenchmark.scala 0.00% <ø> (ø)
.../main/scala-2/io/circe/benchmark/ExampleData.scala 100.00% <ø> (ø)
.../scala-2/io/circe/benchmark/FoldingBenchmark.scala 89.65% <ø> (ø)
...mark/src/main/scala-2/io/circe/benchmark/Foo.scala 100.00% <ø> (ø)
...o/circe/benchmark/GenericDerivationBenchmark.scala 100.00% <ø> (ø)
...-2/io/circe/benchmark/InstantiationBenchmark.scala 73.33% <ø> (ø)
...ala-2/io/circe/benchmark/JsonObjectBenchmark.scala 100.00% <ø> (ø)
...ala-2/io/circe/benchmark/MapDecoderBenchmark.scala 0.00% <ø> (ø)
...-2/io/circe/benchmark/NumberParsingBenchmark.scala 100.00% <ø> (ø)
.../scala-2/io/circe/benchmark/PointerBenchmark.scala 100.00% <ø> (ø)
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 12bfdbf...21fb4f3. Read the comment docs.

@armanbilge
Copy link
Contributor Author

armanbilge commented Jun 19, 2022

@zarthross we cannot update to sbt-git 2.0.0 here, because of the sbt-microsites plugin. As proposed in #1958 (comment) switching to sbt-typelevel-site would solve that issue.

@armanbilge
Copy link
Contributor Author

@zarthross polite bump on this :) build overhauls tend to have large surface area for conflict.

@zarthross
Copy link
Member

I think it looks fine. I'll have @zmccoy look over it too before we merge it in. Thanks for taking this on @armanbilge, and sorry for the late review.

@armanbilge If you still feel like switching this over to sbt-typelevel-site i would welcome the change.

@armanbilge
Copy link
Contributor Author

No problem, thanks for reviewing! Sure, I'll work on the site in a follow-up PR :)

Copy link
Member

@zmccoy zmccoy left a comment

Choose a reason for hiding this comment

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

This looks great! Sorry this took a bit for some attention to get to it.

Thank you for doing this!

@zarthross zarthross merged commit 56de53a into circe:main Jul 11, 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.

4 participants