-
-
Notifications
You must be signed in to change notification settings - Fork 0
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 snapshots with new align #58
Conversation
Signed-off-by: Davide Garolini <dgarolini@gmail.com>
…ineering/scda.test into 135_fix_align_left@57_fix_after_align_left@main
Signed-off-by: Davide Garolini <dgarolini@gmail.com>
default_formatting = list( | ||
all = formatters::fmt_config(align = "left"), | ||
numeric = formatters::fmt_config(align = "center") | ||
), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
took this from @BFalquet. If we need to set it up everywhere better to start now ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, is this a default @barnett11? I cannot find the issue where you talked about this, sorry
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was the discussion, left alignment is needed for this template
https://github.com/insightsengineering/sme-tasks/issues/619
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shajoezhu fyi
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to vbump formatters
Co-authored-by: Joe Zhu <sha.joe.zhu@gmail.com> Signed-off-by: Davide Garolini <dgarolini@gmail.com>
@shajoezhu could you take a look at this again please? |
sorry, I had missed this tag before. @Melkiades , could you please update the code and rerun tests please. thanks! |
Signed-off-by: Davide Garolini <dgarolini@gmail.com>
ready @shajoezhu! :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! thanks @Melkiades
fixes insightsengineering/sme-tasks#57 to be merged only after insightsengineering/rlistings#136