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

Create datum-concepts.md #29

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

thomasdaly5
Copy link

Trying to provide an overview of Datum Concepts for new users.

Tyring to provide an overview of Datum Concepts for new users.

Signed-off-by: Tom Daly <[email protected]>
@joshlreese joshlreese self-requested a review March 7, 2025 18:09
Copy link
Contributor

@joshlreese joshlreese left a comment

Choose a reason for hiding this comment

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

Thanks for bootstrapping this document, it's a great start.

I've made a handful of recommendations and came across a couple spelling errors.

Copy link
Contributor

Choose a reason for hiding this comment

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

Styling request - We don't have a linter for this yet, but could we keep line length at 80 characters to help with reviewing content? It'd allow for more targeted comments and recommended changes.

If helpful, I use the Rewrap plugin in VSCode to help me with this - it allows me to select a block of text and wrap it at an arbitrary length. Another helpful plugin to use is the Markdown All in One plugin package.

thomasdaly5 and others added 5 commits March 10, 2025 15:55
Signed-off-by: Tom Daly <[email protected]>
@thomasdaly5 thomasdaly5 requested a review from joshlreese March 21, 2025 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants