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 renderer/tile docs and type comments #267

Merged
merged 2 commits into from
Dec 26, 2021
Merged

Update renderer/tile docs and type comments #267

merged 2 commits into from
Dec 26, 2021

Conversation

chrahunt
Copy link
Owner

@chrahunt chrahunt commented Dec 26, 2021

Fills in some details gathered while looking through code in consideration of #257.

See individual commits for details.

Helps autocomplete and find usages, making it easier to debug. Tiles and
the renderer modules are pretty closely related, so there's some overlap
in the docs, and some renderer-related info is mentioned in the tiles
module so it is easier to understand.
No values changed, just formatting.

This kind of formatting can create overhead and push away outside
contributors just trying to make minimal changes.
@chrahunt chrahunt marked this pull request as ready for review December 26, 2021 19:34
@chrahunt chrahunt merged commit 6933a63 into master Dec 26, 2021
@chrahunt chrahunt deleted the renderer-docs branch December 26, 2021 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant