Skip to content

Commit

Permalink
Merge pull request #777 from Dwolla/remove-redundant-code
Browse files Browse the repository at this point in the history
Remove redundant code
  • Loading branch information
bpholt authored Sep 19, 2024
2 parents eb41820 + a157b24 commit b05db67
Show file tree
Hide file tree
Showing 10 changed files with 61 additions and 337 deletions.
62 changes: 48 additions & 14 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,10 @@ jobs:
os: [ubuntu-latest]
scala: [3, 2.13, 2.12]
java: [temurin@8, temurin@11]
project: [fs2-awsJS, fs2-awsJVM]
exclude:
- project: fs2-awsJS
java: temurin@11
runs-on: ${{ matrix.os }}
timeout-minutes: 60
steps:
Expand Down Expand Up @@ -71,21 +75,21 @@ jobs:
- name: Check that workflows are up to date
run: sbt githubWorkflowCheck

- run: sbt '++ ${{ matrix.scala }}' test mimaReportBinaryIssues doc
- run: sbt 'project ${{ matrix.project }}' '++ ${{ matrix.scala }}' test mimaReportBinaryIssues doc

- name: Make target directories
if: github.event_name != 'pull_request' && (startsWith(github.ref, 'refs/tags/v') || github.ref == 'refs/heads/3.x')
run: mkdir -p core/js/target core/jvm/target aws-java-sdk2/target project/target
run: mkdir -p core/js/target core/jvm/target project/target

- name: Compress target directories
if: github.event_name != 'pull_request' && (startsWith(github.ref, 'refs/tags/v') || github.ref == 'refs/heads/3.x')
run: tar cf targets.tar core/js/target core/jvm/target aws-java-sdk2/target project/target
run: tar cf targets.tar core/js/target core/jvm/target project/target

- name: Upload target directories
if: github.event_name != 'pull_request' && (startsWith(github.ref, 'refs/tags/v') || github.ref == 'refs/heads/3.x')
uses: actions/upload-artifact@v4
with:
name: target-${{ matrix.os }}-${{ matrix.java }}-${{ matrix.scala }}
name: target-${{ matrix.os }}-${{ matrix.java }}-${{ matrix.scala }}-${{ matrix.project }}
path: targets.tar

publish:
Expand Down Expand Up @@ -133,32 +137,62 @@ jobs:
if: matrix.java == 'temurin@11' && steps.setup-java-temurin-11.outputs.cache-hit == 'false'
run: sbt +update

- name: Download target directories (3)
- name: Download target directories (3, fs2-awsJS)
uses: actions/download-artifact@v4
with:
name: target-${{ matrix.os }}-${{ matrix.java }}-3
name: target-${{ matrix.os }}-${{ matrix.java }}-3-fs2-awsJS

- name: Inflate target directories (3)
- name: Inflate target directories (3, fs2-awsJS)
run: |
tar xf targets.tar
rm targets.tar
- name: Download target directories (2.13)
- name: Download target directories (3, fs2-awsJVM)
uses: actions/download-artifact@v4
with:
name: target-${{ matrix.os }}-${{ matrix.java }}-2.13
name: target-${{ matrix.os }}-${{ matrix.java }}-3-fs2-awsJVM

- name: Inflate target directories (2.13)
- name: Inflate target directories (3, fs2-awsJVM)
run: |
tar xf targets.tar
rm targets.tar
- name: Download target directories (2.12)
- name: Download target directories (2.13, fs2-awsJS)
uses: actions/download-artifact@v4
with:
name: target-${{ matrix.os }}-${{ matrix.java }}-2.12
name: target-${{ matrix.os }}-${{ matrix.java }}-2.13-fs2-awsJS

- name: Inflate target directories (2.12)
- name: Inflate target directories (2.13, fs2-awsJS)
run: |
tar xf targets.tar
rm targets.tar
- name: Download target directories (2.13, fs2-awsJVM)
uses: actions/download-artifact@v4
with:
name: target-${{ matrix.os }}-${{ matrix.java }}-2.13-fs2-awsJVM

- name: Inflate target directories (2.13, fs2-awsJVM)
run: |
tar xf targets.tar
rm targets.tar
- name: Download target directories (2.12, fs2-awsJS)
uses: actions/download-artifact@v4
with:
name: target-${{ matrix.os }}-${{ matrix.java }}-2.12-fs2-awsJS

- name: Inflate target directories (2.12, fs2-awsJS)
run: |
tar xf targets.tar
rm targets.tar
- name: Download target directories (2.12, fs2-awsJVM)
uses: actions/download-artifact@v4
with:
name: target-${{ matrix.os }}-${{ matrix.java }}-2.12-fs2-awsJVM

- name: Inflate target directories (2.12, fs2-awsJVM)
run: |
tar xf targets.tar
rm targets.tar
Expand Down Expand Up @@ -234,5 +268,5 @@ jobs:
- name: Submit Dependencies
uses: scalacenter/sbt-dependency-submission@v2
with:
modules-ignore: fs2-aws_3 fs2-aws_2.13 fs2-aws_2.12
modules-ignore: fs2-awsnative_3 fs2-awsnative_2.13 fs2-awsnative_2.12 fs2-awsjs_3 fs2-awsjs_2.13 fs2-awsjs_2.12 fs2-awsjvm_3 fs2-awsjvm_2.13 fs2-awsjvm_2.12
configs-ignore: test scala-tool scala-doc-tool test-internal
23 changes: 9 additions & 14 deletions .mergify.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,22 +12,17 @@ pull_request_rules:
- or:
- body~=labels:.*early-semver-patch
- body~=labels:.*early-semver-minor
- status-success=Build and Test (ubuntu-latest, 3, temurin@8)
- status-success=Build and Test (ubuntu-latest, 3, temurin@11)
- status-success=Build and Test (ubuntu-latest, 2.13, temurin@8)
- status-success=Build and Test (ubuntu-latest, 2.13, temurin@11)
- status-success=Build and Test (ubuntu-latest, 2.12, temurin@8)
- status-success=Build and Test (ubuntu-latest, 2.12, temurin@11)
- status-success=Build and Test (ubuntu-latest, 3, temurin@8, fs2-awsJS)
- status-success=Build and Test (ubuntu-latest, 3, temurin@8, fs2-awsJVM)
- status-success=Build and Test (ubuntu-latest, 3, temurin@11, fs2-awsJVM)
- status-success=Build and Test (ubuntu-latest, 2.13, temurin@8, fs2-awsJS)
- status-success=Build and Test (ubuntu-latest, 2.13, temurin@8, fs2-awsJVM)
- status-success=Build and Test (ubuntu-latest, 2.13, temurin@11, fs2-awsJVM)
- status-success=Build and Test (ubuntu-latest, 2.12, temurin@8, fs2-awsJS)
- status-success=Build and Test (ubuntu-latest, 2.12, temurin@8, fs2-awsJVM)
- status-success=Build and Test (ubuntu-latest, 2.12, temurin@11, fs2-awsJVM)
actions:
merge: {}
- name: Label aws-java-sdk2 PRs
conditions:
- files~=^aws-java-sdk2/
actions:
label:
add:
- aws-java-sdk2
remove: []
- name: Label core PRs
conditions:
- files~=^core/
Expand Down
81 changes: 0 additions & 81 deletions aws-java-sdk2/src/main/scala/com/dwolla/fs2aws/kms/KmsAlg.scala

This file was deleted.

67 changes: 0 additions & 67 deletions aws-java-sdk2/src/main/scala/com/dwolla/fs2aws/package.scala

This file was deleted.

27 changes: 3 additions & 24 deletions build.sbt
Original file line number Diff line number Diff line change
Expand Up @@ -13,14 +13,9 @@ ThisBuild / developers := List(
)
ThisBuild / tlBaseVersion := "3.0"
ThisBuild / tlCiReleaseBranches := Seq("3.x")
ThisBuild / tlSonatypeUseLegacyHost := true
ThisBuild / sonatypeCredentialHost := xerial.sbt.Sonatype.sonatypeLegacy
ThisBuild / crossScalaVersions := Seq("3.3.3", "2.13.14", "2.12.20")
ThisBuild / scalaVersion := crossScalaVersions.value.head
ThisBuild / startYear := Option(2018)
ThisBuild / tlMimaPreviousVersions ++= {
if (tlIsScala3.value) Set.empty
else Set("3.0.0-RC1")
}

ThisBuild / githubWorkflowBuild := Seq(WorkflowStep.Sbt(List("test", "mimaReportBinaryIssues", "doc")))
ThisBuild / githubWorkflowJavaVersions := Seq(JavaSpec.temurin("8"), JavaSpec.temurin("11"))
Expand All @@ -43,21 +38,5 @@ lazy val `fs2-utils` = crossProject(JSPlatform, JVMPlatform)
),
)

lazy val `fs2-aws-java-sdk2` = project
.in(file("aws-java-sdk2"))
.settings(
description := "Utility classes for interacting with the V2 AWS Java SDKs from Scala using fs2",
libraryDependencies ++= {
Seq(
"co.fs2" %% "fs2-reactive-streams" % fs2Version,
"org.typelevel" %% "cats-tagless-core" % "0.16.2",
"org.scala-lang.modules" %% "scala-collection-compat" % "2.12.0",
"software.amazon.awssdk" % "kms" % "2.25.70" % Provided,
)
},
)

lazy val `fs2-aws` = project
.in(file("."))
.aggregate(`fs2-utils`.jvm, `fs2-utils`.js, `fs2-aws-java-sdk2`)
.enablePlugins(NoPublishPlugin)
lazy val `fs2-aws` = tlCrossRootProject
.aggregate(`fs2-utils`.jvm, `fs2-utils`.js)

This file was deleted.

Loading

0 comments on commit b05db67

Please sign in to comment.