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

create the infrastructure to use custom tic.R #85

Closed
maelle opened this issue Nov 20, 2018 · 5 comments
Closed

create the infrastructure to use custom tic.R #85

maelle opened this issue Nov 20, 2018 · 5 comments
Assignees

Comments

@maelle
Copy link
Contributor

maelle commented Nov 20, 2018

related to many issues. It probably just means wrapping travis::use_tic() and then replacing the tic.R with our template(s).

  • For packages, we want the regular package tic.R + coverage stuff that might be coveralls (so not included by default in the regular package tic.R) + covrpage + maybe project health Run project health stuff on Travis #81

  • For analysis, we want a deploy step rendering all reports of a folder into output, deploying output to a branch. Cf Analysis: Add production scripts #5

@maelle maelle self-assigned this Nov 21, 2018
@maelle
Copy link
Contributor Author

maelle commented Nov 21, 2018

Replace usethis::use_travis() with pieces of it only

  • travis_enable()
  • use_travis_yml() (unless we need a special one)
  • copy our own custom tic.R depending on project type (for packages, copy pRojects setup)
  • use_travis_deploy() except for basic projects, that'll get no deploy maybe? Or maybe they'll do?
  • travis_set_pat()

@maelle
Copy link
Contributor Author

maelle commented Nov 21, 2018

and do all of this if the session is interactive, do nothing and message if not.

@maelle
Copy link
Contributor Author

maelle commented Nov 22, 2018

it'd be good to have the information on what type of project the project is without having to pass that as an argument. Maybe each function creating a project could

  • create a file .project_function at the beginning and in it would be the function name
  • delete it at the end
    this way one could keep not having the path/project type to setup_repo()

maelle added a commit that referenced this issue Nov 22, 2018
@maelle
Copy link
Contributor Author

maelle commented Nov 22, 2018

Maybe basic projects don't need any Travis stuff 🤔 Of course they could use the project health report, but not sure it's useful.

  • packages: check, deploy docs website, covrpage report, deploy project health report.
  • training: deploy the material on a branch, maybe project health report.
  • analysis: deploy the output on a branch, maybe project health report.

@maelle maelle mentioned this issue Nov 26, 2018
@stephlocke
Copy link
Member

sure, that works

@maelle maelle mentioned this issue Jan 25, 2019
Merged
@maelle maelle closed this as completed in 54e786e Feb 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants