-
Notifications
You must be signed in to change notification settings - Fork 681
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
[Docs] Publish Code Style Guide #2694
Comments
@SmritiSatyanV, could you link the google doc we have? Should we have the page in the community tab of our docs? The task here is to convert the google doc to a flyte doc page. |
https://docs.google.com/document/d/1rhEZqd3Ae8Y9HUxg7yF6LarHrfIHKnPyeke8aMSPgis/edit?usp=sharing - here's the google doc |
The user specifies code guide here. What we have is a documentation style guide. I think we need to have a separate code style guide.. |
Oh, okay. Do we have an issue for porting the doc style guide to our docs? |
Hey, I would like to work on this issue. |
If I'm right I just have to convert the google doc to a flyte doc page ? |
Fixes: flyteorg#2694 Signed-off-by: Akarsh Jain <[email protected]>
Hello 👋, This issue has been inactive for over 9 months. To help maintain a clean and focused backlog, we'll be marking this issue as stale and will close the issue if we detect no activity in the next 7 days. Thank you for your contribution and understanding! 🙏 |
Hello 👋, This issue has been inactive for over 9 months and hasn't received any updates since it was marked as stale. We'll be closing this issue for now, but if you believe this issue is still relevant, please feel free to reopen it. Thank you for your contribution and understanding! 🙏 |
Description
It is great for a project to have consistent code formatting/style. This is often accomplished by a linting, and as part of pre-commit tests. But, it would also be helpful if a styleguide were published to make the choices/opinions explicit.
Are you sure this issue hasn't been raised already?
Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: