Skip to content

Conversation

@villegar
Copy link
Contributor

@villegar villegar commented Sep 2, 2025

  • Reconfigured script to push changes to testStatus on pull request

@StuartWheater
Copy link
Member

Should we aim to have this in RC5?

@codecov-commenter
Copy link

codecov-commenter commented Sep 2, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.42%. Comparing base (e6c3bcb) to head (f48889b).
⚠️ Report is 1 commits behind head on v6.3.4-dev.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           v6.3.4-dev     #425   +/-   ##
===========================================
  Coverage       29.42%   29.42%           
===========================================
  Files             138      138           
  Lines            5729     5729           
===========================================
  Hits             1686     1686           
  Misses           4043     4043           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@villegar
Copy link
Contributor Author

villegar commented Sep 3, 2025

This pull request seems like a bunch of unnecessary small changes, but I was testing different ways to get to the same result now that dsBase is not on CRAN. So, if you would rather avoid merging all these changes, then perhaps we can ignore and just update the workflow file with the following bits:

Add in line 62:

          - name: Install dsBase
          run: |
              Rscript -e 'install.packages(c("RANN", "stringr", "lme4", "dplyr", "reshape2", "polycor", "gamlss", "gamlss.dist", "mice", "childsds", "usethis", "devtools"), dependencies = TRUE)'
              R CMD INSTALL ./dsBase   

Line 70:

           dependencies: 'c("Imports")'

Line 166:

          # Reconfigure remote to use GitHub token for authentication
          git remote set-url origin https://x-access-token:${{ secrets.GITHUB_TOKEN }}@github.com/${{ env.REPO_OWNER }}/testStatus.git
          git checkout master
          git pull
          git pull origin master

Line 190:

           git commit -m "Auto test for ${{ env.PROJECT_NAME }}/${{ env.BRANCH_NAME }} @ ${{ env.WORKFLOW_ID }}" || echo "No changes to commit"
          git push origin master

@OllyButters OllyButters merged commit 15c69a4 into datashield:v6.3.4-dev Sep 5, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants