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.

Sign up for GitHub

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

Jump to bottom

Document how users can use linting tools instead of kedro lint #1620

Closed
merelcht opened this issue Jun 14, 2022 · 0 comments · Fixed by #1904
Closed

Document how users can use linting tools instead of kedro lint #1620

merelcht opened this issue Jun 14, 2022 · 0 comments · Fixed by #1904
Assignees
@ankatiyar
Labels
Component: Documentation 📄 Issue/PR for markdown and API documentation
Milestone
Remove developmen...

Comments

@merelcht
Copy link
Member

merelcht commented Jun 14, 2022 •
edited
Loading

In #1293 we decided to remove the kedro lint command.

We need to update our documentation to remove any suggestions of kedro lint and instead write instructions for our users on how to use the linting tools that kedro lint combined: black, flake8 and isort. We can also recommend them to setup pre-commit hooks to run linting automatically, as some users reported using already.

Also , explain how users should setup configuration for the linting tools.
@merelcht merelcht added this to the Cleanup CLI commands milestone Jun 14, 2022
@merelcht merelcht added the Component: Documentation 📄 Issue/PR for markdown and API documentation label Jun 14, 2022
@merelcht merelcht mentioned this issue Jun 16, 2022
Closed
@jmholzer jmholzer mentioned this issue Sep 14, 2022
Closed
@merelcht merelcht moved this to To Do in Kedro Framework Sep 26, 2022
@ankatiyar ankatiyar self-assigned this Sep 26, 2022
@ankatiyar ankatiyar moved this from To Do to In Progress in Kedro Framework Sep 26, 2022
@ankatiyar ankatiyar moved this from In Progress to To Do in Kedro Framework Sep 26, 2022
@noklam noklam mentioned this issue Sep 27, 2022
Merged
5 tasks
@ankatiyar ankatiyar moved this from To Do to In Progress in Kedro Framework Sep 30, 2022
This was referenced Oct 3, 2022
Merged
Closed
@ankatiyar ankatiyar mentioned this issue Oct 5, 2022
Merged
5 tasks
@ankatiyar ankatiyar moved this from In Progress to In Review in Kedro Framework Oct 6, 2022
Repository owner moved this from In Review to Done in Kedro Framework Oct 10, 2022
@merelcht merelcht mentioned this issue Nov 28, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ankatiyar ankatiyar

Labels
Component: Documentation 📄 Issue/PR for markdown and API documentation
Projects
Kedro Framework
Archived in project
Milestone
Remove development related CLI commands
Development

Successfully merging a pull request may close this issue.

Document how users can use linting tools instead of kedro lint kedro-org/kedro
2 participants
@merelcht @ankatiyar