docs icon indicating copy to clipboard operation
docs copied to clipboard

Improve guide on working with the Paper source code

Open Strokkur424 opened this issue 11 months ago • 3 comments

I think it could be nice to include more information on how to work with Paper's source code. This would include not only information for contributors, kinda how we already have the Events page, but also information regarding how to correctly fork Paper (use paperweight to do so) and maintain your own fork could be part of that category.

The following pages are what came to mind first. I wish to ask for anybody to suggest more pages or changes to these:

Page name Page content
Contributing to Paper How to open a PR, what to watch out for, basically CONTRIBUTING.md in the main repo
Styling guidelines Perhaps a dedicated page to certain guidelines, like imports, // Paper comments, and similar
Forking Paper A page explaining how to create and maintain your own fork of Paper using paperweight

Opening this as an issue first, since I want to have some input whether anybody thinks this is really needed in the first place. We already have this one Events contribution page without much context around it, so I think it could be good to extend onto that.

Strokkur424 avatar Jan 11 '25 16:01 Strokkur424

Contributing to Paper should mention that bigger changes should be discussed in an issue first. not sure if a forking paper page is needed. its an advanced topic and we don't really provide support for that, other than throwing the tooling and example repo at people.

MiniDigger avatar Jan 12 '25 09:01 MiniDigger

That's a very good point. It should avoid that people do 2k line PRs and end up being rejected because it doesn't fit the scope of an API.

Furthermore, the forking page could just be a very quick reference to where you can find information about it, doesn't have to be a full-on guide. Just so that we have something to throw at people who ask in #paper-dev on how to create a fork. With all the tooling in the right place. I think people who really need to fork, will somehow figure it out

Strokkur424 avatar Jan 12 '25 11:01 Strokkur424

It should avoid that people do 2k line PRs

It won't avoid it as not all contributors will read that guide. We can only do as much to minimize the amount of huge PRs that get rejected in the end.

A how to create a Paper fork page sounds nice in general.

Leguan16 avatar Jan 12 '25 12:01 Leguan16