-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
pkgdown layout is broken #1359
Comments
Hey @cicdguy is this something close to what you have just fixed? |
I think it might be related. I'll rerun the workflow to see what it is. |
I think it is related. I run the Have a look at this 9745ffa#diff-539e48ef777e8ae44686d203f7a8d44099150953e61b552694032fe7a3a428ba. This is the last commit on main from 5h ago so it includes yesterday fix.
In other words: line 78 belongs to Reports and we should be right before it. On main we have: That comment should be inside WDYT? @cicdguy |
Ah yes, it is indeed that. The bug still exists. Working on a more robust fix. |
I have 2 proposals:
Thoughts? |
|
I'll schedule a call for early next week and we can go over both options. I can explain the situation better that way. |
Visible only for
main
: https://insightsengineering.github.io/teal/main/might be related: insightsengineering/r-pkgdown-multiversion#22
The text was updated successfully, but these errors were encountered: