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

AlignSettingsSection:skip_documentation has wrong default value #722

Open
tkopra opened this issue Dec 9, 2024 · 2 comments
Open

AlignSettingsSection:skip_documentation has wrong default value #722

tkopra opened this issue Dec 9, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@tkopra
Copy link

tkopra commented Dec 9, 2024

Robotidy 4.9.0 was supposed to just allow skip formatting documentation in AlignSettingsSection transformer. It however does the skipping now by default. Thus I needed to set AlignSettingsSection:skip_documentation=False to prevent that in my old project.

@bhirsz
Copy link
Member

bhirsz commented Dec 9, 2024

Do you use global --skip-documentation or :skip_documentation in any other transformer? I still see it as False by default in the source code. But I might omit something, or it could bleed from some other place to AlignSettingsSection.

@bhirsz bhirsz added the bug Something isn't working label Dec 9, 2024
@tkopra
Copy link
Author

tkopra commented Dec 9, 2024

Yes actually using skip-documentation = true in "global" configuration. Idea has been to avoid any other transformations in the Documentation. So in that sense this is logical now, just having impact on any old projects configured in this way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants