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

Deprecate BS3 #2545

Merged
merged 3 commits into from
May 16, 2024
Merged

Deprecate BS3 #2545

merged 3 commits into from
May 16, 2024

Conversation

hadley
Copy link
Member

@hadley hadley commented May 14, 2024

No description provided.

Copy link

github-actions bot commented May 14, 2024

Copy link
Collaborator

@jayhesselberth jayhesselberth left a comment

Choose a reason for hiding this comment

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

Looks good to me

NEWS.md Outdated Show resolved Hide resolved
@@ -35,6 +35,13 @@ pkgdown_sitrep <- function(pkg = ".") {
cli::cli_rule("Sitrep")

pkg <- as_pkgdown(pkg)
if (pkg$bs_version == 3) {
cli::cli_inform(c(
Copy link
Collaborator

Choose a reason for hiding this comment

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

this will be visible in CI logs? should there be more info, a new short blog post maybe?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm going to think more about how to expose the sitrep more clearly in automated checks.

@hadley hadley merged commit 6e6336e into main May 16, 2024
14 checks passed
@hadley hadley deleted the bs3-dep branch May 16, 2024 15:07
SebKrantz pushed a commit to SebKrantz/pkgdown that referenced this pull request Jun 1, 2024
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.

3 participants