From 0b9200bc9652f4c0d401618f0e34236a2c3d78b8 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 23 Jul 2026 20:59:13 +0000 Subject: [PATCH 1/4] Initial plan From c819e7e661b1712a4d5c7b65bcf56fbfbadd5fea Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 23 Jul 2026 21:02:10 +0000 Subject: [PATCH 2/4] fix(ci): install quantstrat deps without unavailable suggests --- .../workflows/reverse-dependency-check.yaml | 105 ++++++++++++++++++ 1 file changed, 105 insertions(+) create mode 100644 .github/workflows/reverse-dependency-check.yaml diff --git a/.github/workflows/reverse-dependency-check.yaml b/.github/workflows/reverse-dependency-check.yaml new file mode 100644 index 0000000..c850bbc --- /dev/null +++ b/.github/workflows/reverse-dependency-check.yaml @@ -0,0 +1,105 @@ +name: Reverse dependency checks + +on: + workflow_dispatch: + push: + pull_request: + +permissions: + contents: read + +concurrency: + group: reverse-deps-${{ github.workflow }}-${{ github.ref }} + cancel-in-progress: true + +jobs: + reverse-dependencies: + name: blotter and quantstrat + runs-on: ubuntu-latest + + steps: + - name: Check out FinancialInstrument + uses: actions/checkout@v4 + + - name: Set up R + uses: r-lib/actions/setup-r@v2 + with: + r-version: release + use-public-rspm: true + + - name: Install test tools and FinancialInstrument dependencies + uses: r-lib/actions/setup-r-dependencies@v2 + with: + extra-packages: | + any::rcmdcheck + any::remotes + + - name: Install current FinancialInstrument + run: R CMD INSTALL . + + - name: Clone downstream packages + run: | + mkdir -p revdeps + + git clone --depth 1 \ + https://github.com/braverock/blotter.git \ + revdeps/blotter + + git clone --depth 1 \ + https://github.com/braverock/quantstrat.git \ + revdeps/quantstrat + + - name: Install blotter dependencies + shell: Rscript {0} + run: | + pak::local_install_dev_deps( + "revdeps/blotter" + ) + + - name: Reinstall current FinancialInstrument + run: R CMD INSTALL . + + - name: Check blotter + shell: Rscript {0} + run: | + result <- rcmdcheck::rcmdcheck( + "revdeps/blotter", + args = c( + "--no-manual", + "--no-vignettes" + ), + error_on = "error" + ) + + print(result) + + - name: Install checked blotter + run: R CMD INSTALL revdeps/blotter + + - name: Install quantstrat dependencies + shell: Rscript {0} + run: | + remotes::install_deps( + "revdeps/quantstrat", + dependencies = c("Depends", "Imports", "LinkingTo"), + upgrade = "never" + ) + + - name: Reinstall tested package chain + run: | + R CMD INSTALL . + R CMD INSTALL revdeps/blotter + + - name: Check quantstrat + shell: Rscript {0} + run: | + result <- rcmdcheck::rcmdcheck( + "revdeps/quantstrat", + args = c( + "--no-manual", + "--no-vignettes" + ), + error_on = "error" + ) + + print(result) From 15147393e45632e74e00eaa5ba1923c6e540c958 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 23 Jul 2026 21:03:14 +0000 Subject: [PATCH 3/4] chore(ci): make reverse-dependency tooling explicit --- .github/workflows/reverse-dependency-check.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/reverse-dependency-check.yaml b/.github/workflows/reverse-dependency-check.yaml index c850bbc..ba335fa 100644 --- a/.github/workflows/reverse-dependency-check.yaml +++ b/.github/workflows/reverse-dependency-check.yaml @@ -32,6 +32,7 @@ jobs: with: extra-packages: | any::rcmdcheck + any::pak any::remotes - name: Install current FinancialInstrument @@ -76,7 +77,7 @@ jobs: - name: Install checked blotter run: R CMD INSTALL revdeps/blotter - - name: Install quantstrat dependencies + - name: Install quantstrat core dependencies shell: Rscript {0} run: | remotes::install_deps( From 2d07cbe2b3f29d2ab25ecb79a323221d7b4ca163 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 23 Jul 2026 21:16:50 +0000 Subject: [PATCH 4/4] fix(ci): avoid GitHub auth path in quantstrat dependency install --- .github/workflows/reverse-dependency-check.yaml | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/.github/workflows/reverse-dependency-check.yaml b/.github/workflows/reverse-dependency-check.yaml index ba335fa..e44fa48 100644 --- a/.github/workflows/reverse-dependency-check.yaml +++ b/.github/workflows/reverse-dependency-check.yaml @@ -33,7 +33,6 @@ jobs: extra-packages: | any::rcmdcheck any::pak - any::remotes - name: Install current FinancialInstrument run: R CMD INSTALL . @@ -80,11 +79,15 @@ jobs: - name: Install quantstrat core dependencies shell: Rscript {0} run: | - remotes::install_deps( - "revdeps/quantstrat", - dependencies = c("Depends", "Imports", "LinkingTo"), - upgrade = "never" - ) + pak::pkg_install(c( + "quantmod", + "xts", + "foreach", + "iterators", + "zoo", + "TTR", + "MASS" + )) - name: Reinstall tested package chain run: |