Skip to content
Grav 2.0 is officially stable. Read the announcement →

Community guidelines

Please keep discussions civil and on-topic. Repeated violations may lead to a temporary ban.

Archive

On Github, is forking a theme bad?

Started by Muut Archive 12 years ago · 1 replies · 441 views
12 years ago

I'm messing around with Grav and really liking the project, but I'm really new to GitHub.

So, is it seen as a bad thing to fork one of the current grav themes to work on edits I want to try?

Is there a better strategy?

Just looking for any of those "common courtesy" rules I might not know just yet and don't want to stay ignorant to. :)

12 years ago

That makes perfectly sense and GitHub fork is meant exactly for that. The fork allows you to create a copy of a project you like under your own user. At that point any change you do, you can commit and push to your own repository.
You can then decide to contribute back via GitHub and through pull requests or even start a whole new project of your own.
GitHub is really great about this, it always shows you what the parent you forked from is, it lets you compare your changes with the parent one, lets you keep in sync all the changes from the main repo to yours and so own.

When you see pull requests going under the main project, that's because someone forked the project, created a fix and then pull requested back. Usually fixes and patches are done through branches to avoid polluting the develop branch where you might have other changes, but this is a different story.

You can read more about fork from GitHub itself, b ut the bottom line is: go ahead, fork and experiment, that's one of the many reasons we decided to put everything on GitHub in the first place!

Suggested topics

Topic Participants Replies Views Activity
Archive · by Deleted User, 9 years ago
0 1277 9 years ago
Archive · by Muut Archive, 9 years ago
2 885 9 years ago
Archive · by Muut Archive, 9 years ago
2 4014 9 years ago
Archive · by Muut Archive, 9 years ago
1 2890 9 years ago
Archive · by Muut Archive, 9 years ago
3 1074 9 years ago