Lorenz Walthert

Results 539 comments of Lorenz Walthert

Thanks @gravesti. I spend at least 5h working on this PR (and some related issues that came up due to it) over the last 3 days. -.- I see that...

@gravesti this will be available when there is a new release (i.e. new tag on `main`). Or if you use a hash pointing to HEAD (e.g. [59981a7](https://github.com/lorenzwalthert/precommit/commit/59981a7fdffc3ece2e690ad70742e5f5ca5a9d86)) instead of a...

Ok, good to know. But since we want to be style guide compliant, I think we’ll eventually also implement that functionality.

No, kind of waiting for the respective tidyverse style issue to be addressed.

@IndrajeetPatil do you merge manually? I keep getting notifications and I think I don’t want them I until this is ready for review. Converting to draft now.

@Robinlovelace if you want this feature, please send a PR.

~Fixed with https://github.com/aws/sagemaker-training-toolkit/pull/156 I think.~

This is done by styler by default when the [alignment Definition](https://styler.r-lib.org/articles/detect-alignment.html) is matched. We are working on detecting {datapasta} tribble style: https://github.com/r-lib/styler/issues/753

Did you consider [Local Mode](https://github.com/aws-samples/amazon-sagemaker-local-mode)? We use this feature for local prototyping and it works well except some limitations in the context of Pipeline and Experiment integration, e.g. #4114.