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

Add guides for using an FTP and GitHub setup in a fansub group #65

Merged

Conversation

LightArrowsEXE
Copy link
Member

I have no real plans of writing the FTP section since I simply don't know how to set one up, but I did write one for basic GitHub use. This is intended to be eventually referenced to when I write about SubKt, which works best when combined with a Git setup. Not sure if it's a good idea to also include that in this PR, but I'll hear from you if it's better to make a separate PR for that or not.

Copy link
Member

@FichteFoll FichteFoll left a comment

Choose a reason for hiding this comment

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

As I said on Discord, I like the idea of introducing git to sub groups, although it might not be the easiest entry. However, some things will need to be changed before I can merge this, outside of the docs not building.

src/SUMMARY.md Outdated Show resolved Hide resolved
src/subtitling/Group setups/ftp.md Outdated Show resolved Hide resolved
src/subtitling/Group setups/github.md Outdated Show resolved Hide resolved
src/subtitling/Group setups/github.md Outdated Show resolved Hide resolved
src/subtitling/Group setups/github.md Outdated Show resolved Hide resolved
src/subtitling/Group setups/github.md Outdated Show resolved Hide resolved
src/subtitling/Group setups/github.md Outdated Show resolved Hide resolved
src/subtitling/Group setups/github.md Outdated Show resolved Hide resolved
src/subtitling/Group setups/github.md Outdated Show resolved Hide resolved
src/subtitling/Group setups/github.md Outdated Show resolved Hide resolved
@FichteFoll FichteFoll added the content We can do better! label Jun 12, 2021
Copy link
Member Author

@LightArrowsEXE LightArrowsEXE left a comment

Choose a reason for hiding this comment

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

b9c4a8a Bloody hell, it is. >:(
Changing the casing locally does not update it when I push it to GitHub, so idk if you know any fancy tricks to lowercase it.

Co-authored-by: Dave <orangechannel@pm.me>
@FichteFoll
Copy link
Member

Github somehow marked this PR as read despite me not having read it since my last review, so I almost forgot about it. I'll see to it within the week.

src/subtitling/Collaboration/github.md Show resolved Hide resolved
src/SUMMARY.md Show resolved Hide resolved
@FichteFoll FichteFoll merged commit 1796981 into Irrational-Encoding-Wizardry:master Jul 1, 2021
@FichteFoll
Copy link
Member

Merged manually with a few changes on top.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content We can do better!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants