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

make it clear that this repo requires updating and shouldn't be used #90

Merged
merged 1 commit into from
Jan 5, 2021
Merged

make it clear that this repo requires updating and shouldn't be used #90

merged 1 commit into from
Jan 5, 2021

Conversation

mfaani
Copy link
Contributor

@mfaani mfaani commented Dec 29, 2020

While I'm very happy with the days spent learning ruby, rake, rbenv, bundler and how to fix environmental issues. I think being loud and clear that the repo needs an update can save future dev time.

@dnkoutso's comment on the issue I opened:

that probably needs updating to be honest.

Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

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

LGTM! I did similar spinning when I first tried to contribute to CocoaPods.

It would also be great to remove the Rainforest references from https://guides.cocoapods.org/contributing/dev-environment.html. I'll make a PR for that.

@mfaani
Copy link
Contributor Author

mfaani commented Dec 29, 2020

@paulb777 Great idea. I wasn't so sure what exactly to put there instead of a simple clone and bootstrap. Thank you!

@paulb777
Copy link
Member

CocoaPods/guides.cocoapods.org#196

@mfaani
Copy link
Contributor Author

mfaani commented Jan 5, 2021

@paulb777 Do you know what else needs to be done so this can get merged?

@paulb777
Copy link
Member

paulb777 commented Jan 5, 2021

@dnkoutso Can you help here?

@dnkoutso dnkoutso merged commit 1cd6376 into CocoaPods:master Jan 5, 2021
@dnkoutso
Copy link
Contributor

dnkoutso commented Jan 5, 2021

Merged. CI has been broken for a while here. We can fix but not yet.

@mfaani
Copy link
Contributor Author

mfaani commented Jan 5, 2021

@dnkoutso 🎉 🎉 🎉 Tiny but first OSS contribution

@paulb777
Copy link
Member

paulb777 commented Jan 5, 2021

Congrats @prohoney ! 🎉 🎉 🎉

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.

3 participants