[dagster-pipes, docs] apply ruff to docs_snippets #24935
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
danielgafni
force-pushed
the
09-03-_dagster-aws_docs_add_docs_for_pipesemrserverlessclient
branch
from
6e48e07 to
bddf8f1
Compare
danielgafni
force-pushed
the
10-01-_dagster-pipes_docs_apply_ruff_to_docs_snippets
branch
from
6e81958 to
5ea968a
Compare
|
Mostly the changes are harmless, but I've spotted a few docs with multiple pieces of the same code snippet where imports reordering affected the first import appearance. I'm not looking for a full review before before I fix these small issues. @cmpadden @PedramNavid what are your thoughts on this idea in general? |
@danielgafni I am generally in favor of enforcing the same style throughout the code base. So applying our main To be honest, I'm surprised this wasn't already happening. |
|
Yeah I decided to start with Pipes because it's my area of responsibility, but we probably want to extend this practice to all examples and docs at some point. |
| @@ -0,0 +1,163 @@ | |||
| --- | |||
There was a problem hiding this comment.
If possible, could you separate the new guide content from the ruff formatting of docs_snippets?
Summary & Motivation
This PR applies
rufftodocs_snippetsfor Dagster Pipes docs.This is done by adding a
examples/docs_snippets/docs_snippets/guides/dagster/dagster_pipes/ruff.tomlfile extending the mainruffconfig.How I Tested These Changes
Changelog
NOCHANGELOG