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

docs: Stylize announcement, update contributors in README.md #123

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

charlieforward9
Copy link
Contributor

@charlieforward9 charlieforward9 commented May 23, 2024

I contributed #96 and want to put my name on the README for Open Source clout.

Screenshot 2024-05-23 at 2 19 03 AM

With more people using the most current version than the outdated version, it also seemed time to restore the face of the README to a cleaner state, with a description of the package under the title and a formatted alert of the package location change following that.

Important

This should be merged asap 🙏 , thanks @mapbox

Shoutout to Github for these cool stylers

CC: @sheindel

Stylize announcement, update contributors
Improve all the other sections of the README
@charlieforward9 charlieforward9 changed the title Stylize announcement, update contributors in README.md docs: Stylize announcement, update contributors in README.md May 23, 2024
@charlieforward9
Copy link
Contributor Author

@sheindel I would appreciate seeing this merged as it's simply a README modification that improves docs and should not require code review. Its one of my oldest unmerged PRs that I'd like to see finished.

@sheindel
Copy link
Contributor

sheindel commented Nov 12, 2024

I'd love to get these in too, but I was removed by mapbox as being in control of the repository. Presumably my access was removed as part of a regular sweep, but they decided not to reinstate it. @chriswhong may be able to help or delegate this but no guarantees as he is quite busy as well and I don't think mapbox actually relies on this library themselves anymore.

@charlieforward9
Copy link
Contributor Author

charlieforward9 commented Nov 13, 2024

Thanks Stephen,

Chris, TLDR: these are non-breaking stylistic updates to docs that I just want to get off my active PR list. Thank you.

Also happy to maintain this repo if needed.

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.

2 participants