Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Update libraries.md to be more detailed #446

Merged
merged 5 commits into from
Dec 25, 2020
Merged

Update libraries.md to be more detailed #446

merged 5 commits into from
Dec 25, 2020

Conversation

CouldBeThis
Copy link
Contributor

I made instructions for adding libraries more explicit because I got lost in the menus myself.

pictures/animation would be better but I'm not sure how to upload screen caps and also I don't know if that would be useful/welcome.

I made instructions for adding libraries more explicit because i got lost in the menus myself. pictures/animation would be better but i'm not sure how to upload screen caps.
@dkanada
Copy link
Member

dkanada commented Nov 28, 2020

We're fine with images, and they can be added in the images directory and referenced in the markdown files. Here are the linting errors for this pull request, let me know if any of them are unclear.

general/server/libraries.md:12:91 MD034/no-bare-urls Bare URL used [Context: "http://localhost:8096/"]
general/server/libraries.md:13:206 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
general/server/libraries.md:14:30 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
general/server/libraries.md:15:57 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
general/server/libraries.md:17:252 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
general/server/libraries.md:18:302 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

@CouldBeThis
Copy link
Contributor Author

"markdown linting" what a neat concept, I am going to look at it further.

I think I have made the edits requested and put the file in the correct place. If not I will be grateful to know about it so I can do it properly next time.

thank you for your patience

@dkanada dkanada merged commit 241b91d into jellyfin-archive:master Dec 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants