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

ref: prioritize targets over drake #63

Merged
merged 1 commit into from
Apr 30, 2024
Merged

ref: prioritize targets over drake #63

merged 1 commit into from
Apr 30, 2024

Conversation

psanker
Copy link
Member

@psanker psanker commented Apr 30, 2024

Closes #53

Description of changes:

  • Mark targets as Imports and drake as Suggests
  • Change main testing engine to targets

targets is the de-facto workflow execution engine in the R ecosystem
now, with drake being deprecated for a while. For future improvements to
blueprintr, most functionality will be targeted at the targets engine.

Requirements:

  • Does your PR include unit tests?
  • Does your PR pass R CMD check ?
  • If you made user-facing changes, did you update NEWS.md with a new bullet? Be sure to tag your name and relevant issue number like * Made text more readable (@yourname, #1234)

targets is the de-facto workflow execution engine in the R ecosystem
now, with drake being deprecated for a while. For future improvements to
blueprintr, most functionality will be targeted at the targets engine.
@psanker psanker force-pushed the ref/prefer-targets branch from 3ae0423 to 15e3d4b Compare April 30, 2024 02:52
@psanker psanker merged commit b2e4293 into main Apr 30, 2024
5 checks passed
@psanker psanker deleted the ref/prefer-targets branch April 30, 2024 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prioritize {targets}
1 participant