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

Update readme files for consistency/ext. #36

Merged
merged 9 commits into from Sep 6, 2019
Merged

Update readme files for consistency/ext. #36

merged 9 commits into from Sep 6, 2019

Conversation

ghost
Copy link

@ghost ghost commented Sep 1, 2019

I updated a few of the READMEs for consistency and spelling and the like, as well as fixing the README for the images folder.

Thunkstation added 5 commits September 1, 2019 15:08
Minor corrections as well as updates to the resources section
ISO8601 format added for pattern with example given
Corrected a few minor spelling/grammar errors
@roboyoshi
Copy link
Owner

Hey @Thunkstation much appreciated PR. From a quick overview it looks fine. Will go through it again in more detail to review later!

@roboyoshi roboyoshi added enhancement readme An issue that extends readme information for a particular section labels Sep 2, 2019
@roboyoshi roboyoshi changed the title Readme edits Update readme files for consistency/ext. Sep 2, 2019
Copy link
Owner

@roboyoshi roboyoshi left a comment

Choose a reason for hiding this comment

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

Great work! Would like to change a few lines, but otherwise it's fine with me.

README.md Outdated
We aim to create a unified filetree for all kinds of data.
We aim to create a unified filetree for all kinds of data, for making it easier to store, categorise and retrieve it.

This filetree is primarily based on the categorisation of data into directories based on the form of the data involved. For example, if a file is a .mp3 or .wav file, it would then be sorted into the audio top-level section. However, within subsections, the data is usually categorised based on category. If the previously mentioned audio file was a piece of music, it would be sorted into the artists subsection. However, if it was an audiobook, it would be sorted into the books subsection.
Copy link
Owner

Choose a reason for hiding this comment

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

if a file is a .mp3 or .wav file, it would then be sorted into the audio top-level section

That is not the impression one should get here. This repository is aimed to create multiple examples of how one can deploy a branching based filetree for his personal data collection in a curated manner. I've purposely left that one out of the main readme so far, because it wasn't clear what this filetree would become.

Copy link
Author

@ghost ghost Sep 5, 2019

Choose a reason for hiding this comment

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

Okay, thanks for clarifying, this could definitely be expanded with a later version when multiple filetrees are available but for now I'll leave it was before. 👍

README.md Outdated

We aim to create a unified filetree for all kinds of data.
We aim to create a unified filetree for all kinds of data, for making it easier to store, categorise and retrieve it.
Copy link
Owner

@roboyoshi roboyoshi Sep 4, 2019

Choose a reason for hiding this comment

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

My proposal: We aim to create a unified filetree for all kinds of data, which should help in storing, categorising and retrieving.

- https://musicbrainz.org/ - A community-maintained music metadata database.
Copy link
Owner

Choose a reason for hiding this comment

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

👍

@@ -12,7 +12,11 @@ Personal Documents like Letters, Poems, Forms, Bills, etc.

## Patterns

The most common file naming scheme for documents is likely the use of the date in ISO8601 format at the start of the name (i.e YYYY-MM-DD) followed by a dash and then a useful name for the file. These names should be appropriately descriptive and formatted in an easily readable manner that allow it to be distinguished from other documents in the same subsection.
Copy link
Owner

Choose a reason for hiding this comment

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

Well #10 would like to disagree. I agree that the ISO pattern is very common. I'd also like to keep it as a good example. But I'm not sure if it's the most common one. Also would love to see other patterns here. Probably part of a later commit..

Copy link
Author

Choose a reason for hiding this comment

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

I'll change that to "one of the most common", then, generally it's what I see as the most optimal, but there's definitely other options which should be included (like #10)

@@ -1,6 +1,8 @@
## Content

Images may include, but is not limited to:
The images section, most broadly, contains any kind of digital picture.
Copy link
Owner

Choose a reason for hiding this comment

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

any kind of digital picture

IMO this is too restrictive / confusing. For some people (me included) pictures quickly becomes photo.. and I think that link is broken by just using the 'image' as a base and then branch out into pictures/photos/artworks/etc.
I think I had some resources on the exact definitions of picture and photo..

Copy link
Author

@ghost ghost Sep 5, 2019

Choose a reason for hiding this comment

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

Very true, "image" as a term is rather hard to define. I'll leave it as it was before until there's any better definitions available.

@ghost
Copy link
Author

ghost commented Sep 5, 2019

Alright, I'll try and work on some of these tomorrow and see if there's any other small additions/changes that might be good. Thanks for the feedback!

Thunkstation added 4 commits September 6, 2019 18:15
Remove redundant paragraph, and change third paragraph to roboyoshi's suggestion.
Changed most common to one of the most common
@ghost ghost requested a review from roboyoshi September 6, 2019 17:27
@ghost
Copy link
Author

ghost commented Sep 6, 2019

I think I've made all suggested changes, as well as tidying up some of the READMEs.

Copy link
Owner

@roboyoshi roboyoshi left a comment

Choose a reason for hiding this comment

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

LGTM!

@ghost
Copy link
Author

ghost commented Sep 6, 2019

Awesome! Thanks man

@roboyoshi roboyoshi merged commit e8694bc into roboyoshi:master Sep 6, 2019
@ghost ghost deleted the readme-edits branch September 6, 2019 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement readme An issue that extends readme information for a particular section
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant