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

Update demo.Rmd #19

Merged
merged 12 commits into from Dec 28, 2019
Merged

Update demo.Rmd #19

merged 12 commits into from Dec 28, 2019

Conversation

kaslee
Copy link

@kaslee kaslee commented Oct 3, 2019

Updated as per instructions

@codecov
Copy link

codecov bot commented Oct 3, 2019

Codecov Report

Merging #19 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@         Coverage Diff          @@
##           master   #19   +/-   ##
====================================
  Coverage       0%    0%           
====================================
  Files          17    17           
  Lines         149   149           
====================================
  Misses        149   149

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 09c7a7a...41b1ee5. Read the comment docs.

Create new package
You would start by creating a new package. We use the `tempfile()` and `create_package()` functions for this.

The output shows the details of the package created.

```{r }
pkg <- usethis::create_package(tempfile("pkg"))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll change the code later to use "superfrob" as the package name.

pkg
```

Set up development work
Next activity on the list would be to set up this package for development and create a git repository for the same. Following would be the sequence of steps you would most likely follow.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code in this chunk is necessary only for the demonstration here, in a realistic scenario everything will be set up already. Don't need to elaborate further here.


```{r }
setwd(pkg) # FIXME: This shouldn't be necessary
news <- readLines("NEWS.md")
cat(news, sep = "\n")
```

Consolidate messages
You decide that you do not want all those development commit messages to appear in `NEWS.md`. So you decide to change it to something more readable from the release perspective. No problem! You can replace the messages with something more readable as follows
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The idea here is to convert changelog entries to release notes.

@krlmlr krlmlr merged commit 386df6e into cynkra:master Dec 28, 2019
@krlmlr
Copy link
Contributor

krlmlr commented Dec 28, 2019

Thanks!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 28, 2020
krlmlr pushed a commit that referenced this pull request Oct 17, 2022
* Harmonize yaml formatting

Closes #19

* revert changes to matrix section
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants