Skip to content

Commit 58cbbef

Browse files
Move up "add link to blog post" release bullet (#1847)
* Move up release bullet to add link to blog post * add 'preemptive' to blog link bullet Co-authored-by: Davis Vaughan <[email protected]> * silence message in test (unrelated to this PR) * Update snapshot --------- Co-authored-by: Davis Vaughan <[email protected]>
1 parent 395c9d3 commit 58cbbef

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

R/release.R

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,12 +137,12 @@ release_checklist <- function(version, on_cran) {
137137
"Wait for CRAN...",
138138
"",
139139
todo("Accepted :tada:"),
140+
todo("Add preemptive link to blog post in pkgdown news menu", type != "patch"),
140141
todo("`usethis::use_github_release()`"),
141142
todo("`usethis::use_dev_version(push = TRUE)`"),
142143
todo("`usethis::use_news_md()`", !has_news),
143144
todo("Finish blog post", type != "patch"),
144-
todo("Tweet", type != "patch"),
145-
todo("Add link to blog post in pkgdown news menu", type != "patch")
145+
todo("Tweet", type != "patch")
146146
)
147147
}
148148

tests/testthat/_snaps/release.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@
3333
Wait for CRAN...
3434
3535
* [ ] Accepted :tada:
36+
* [ ] Add preemptive link to blog post in pkgdown news menu
3637
* [ ] `usethis::use_github_release()`
3738
* [ ] `usethis::use_dev_version(push = TRUE)`
3839
* [ ] `usethis::use_news_md()`
3940
* [ ] Finish blog post
4041
* [ ] Tweet
41-
* [ ] Add link to blog post in pkgdown news menu
4242

4343
---
4444

@@ -101,12 +101,12 @@
101101
Wait for CRAN...
102102
103103
* [ ] Accepted :tada:
104+
* [ ] Add preemptive link to blog post in pkgdown news menu
104105
* [ ] `usethis::use_github_release()`
105106
* [ ] `usethis::use_dev_version(push = TRUE)`
106107
* [ ] `usethis::use_news_md()`
107108
* [ ] Finish blog post
108109
* [ ] Tweet
109-
* [ ] Add link to blog post in pkgdown news menu
110110

111111
# construct correct revdep bullet
112112

@@ -158,10 +158,10 @@
158158
Wait for CRAN...
159159
160160
* [ ] Accepted :tada:
161+
* [ ] Add preemptive link to blog post in pkgdown news menu
161162
* [ ] `usethis::use_github_release()`
162163
* [ ] `usethis::use_dev_version(push = TRUE)`
163164
* [ ] `usethis::use_news_md()`
164165
* [ ] Finish blog post
165166
* [ ] Tweet
166-
* [ ] Add link to blog post in pkgdown news menu
167167

tests/testthat/test-proj-desc.R

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,10 @@ test_that("proj_desc_field_update() only messages when adding", {
1212

1313
test_that("proj_desc_field_update() works with multiple values", {
1414
create_local_package()
15+
# Add something to begin with
16+
proj_desc_field_update("Config/Needs/foofy", "alfa", append = TRUE)
1517
withr::local_options(list(usethis.quiet = FALSE, crayon.enabled = FALSE))
1618

17-
proj_desc_field_update("Config/Needs/foofy", "alfa", append = TRUE)
1819
expect_snapshot({
1920
proj_desc_field_update("Config/Needs/foofy", c("alfa", "bravo"),
2021
append = TRUE)

0 commit comments

Comments
 (0)