Skip to content
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

Fix the broken CICD #101

Closed
6 tasks done
epijim opened this issue Feb 4, 2021 · 6 comments
Closed
6 tasks done

Fix the broken CICD #101

epijim opened this issue Feb 4, 2021 · 6 comments
Assignees
Labels
bug Something isn't working issue Used for project filtering

Comments

@epijim
Copy link
Collaborator

epijim commented Feb 4, 2021

Issue description

Current CICD doesn't work... fix it.

Steps to reproduce the issue

Push a commit and Github actions doesnt work

What's the expected result?

  • build and deploy docs
  • README.rmd is built automatically
  • check windows
  • check ubuntu
  • check osx
  • calculate code coverage
@epijim epijim added bug Something isn't working issue Used for project filtering labels Feb 4, 2021
@epijim epijim self-assigned this Feb 4, 2021
@epijim
Copy link
Collaborator Author

epijim commented Mar 1, 2021

testthat::expect_equal(list_survival, list_visR) does not equay the same in tests..

@epijim
Copy link
Collaborator Author

epijim commented Mar 2, 2021

added - note fails but it's legitimate (e.g. package fails check rather than CICD). docs building and code coverage point at develop currently- needs to be repointed at master after @rebecca-albrecht and @DaZim make their merge in.

image

@epijim
Copy link
Collaborator Author

epijim commented Mar 3, 2021

Updated it so that the README.md is autobuilt:

      - name: Deploy package
        run: |
          Rscript -e 'rmarkdown::render("README.rmd")'
          git config --local user.email "[email protected]"
          git config --local user.name "GitHub Actions"
          Rscript -e 'pkgdown::deploy_to_branch(new_process = FALSE)'

@epijim
Copy link
Collaborator Author

epijim commented May 20, 2021

rename the branch master to main

@epijim
Copy link
Collaborator Author

epijim commented May 20, 2021

and fastforward to where develop istoday

@epijim
Copy link
Collaborator Author

epijim commented May 21, 2021

CICD appears to be working - e.g. fails are legitimate due to package having CRAN check warnings.

@epijim epijim closed this as completed May 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working issue Used for project filtering
Projects
None yet
Development

No branches or pull requests

1 participant