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

Add best practices guide #257

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

Add best practices guide #257

wants to merge 4 commits into from

Conversation

ruthenian8
Copy link
Member

Description

Add best practices guide

Checklist

  • I have covered the code with tests
  • I have added comments to my code to help others understand it
  • I have updated the documentation to reflect the changes
  • I have performed a self-review of the changes

@RLKRo RLKRo added the documentation Improvements or additions to documentation label Oct 19, 2023
@ruthenian8 ruthenian8 self-assigned this Oct 25, 2023
@ruthenian8 ruthenian8 marked this pull request as ready for review November 24, 2023 09:20
@RLKRo RLKRo requested a review from Ramimashkouk February 7, 2024 16:57
Ramimashkouk
Ramimashkouk reviewed Feb 12, 2024
View reviewed changes
Copy link
Member

@Ramimashkouk Ramimashkouk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many links in this doc won't work until examples PR get merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ramimashkouk Ramimashkouk Ramimashkouk left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@ruthenian8 ruthenian8

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ruthenian8 @Ramimashkouk @RLKRo