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

Add support for readme and changelog pod attributes #658

Merged
merged 1 commit into from
Feb 1, 2021

Conversation

jrullman
Copy link
Contributor

This change adds support and documentation for new optional readme and changelog attributes, which contain URLs pointing to Markdown files. The CocoaPods Metadata Service has already been modified to read these attributes and, if present, download the files and save them for rendering on the pod page (see CocoaPods/cocoapods-metadata-service#6 and CocoaPods/cocoapods-metadata-service#20).

@jrullman
Copy link
Contributor Author

Fix for #661.

@paulb777
Copy link
Member

@dnkoutso Would it be appropriate to add retarget this to 1-10-stable so binary pod publishers could get their CocoaPods web pages fixed sooner?

@paulb777
Copy link
Member

Would you rebase to 1-10-stable so this can go into 1.10.2?

@jrullman jrullman changed the base branch from master to 1-10-stable January 28, 2021 20:01
@jrullman
Copy link
Contributor Author

Done with the rebase.

@dnkoutso
Copy link
Contributor

dnkoutso commented Jan 28, 2021

Should probably get a changelog entry. Could potentially call this 1.11.0 tbh...but it can be considered a "bug fix"

@paulb777
Copy link
Member

Definitely a bug from a binary pod publisher perspective.

@jrullman
Copy link
Contributor Author

I can add an entry to the CHANGELOG.md file, just let me know which section.

@dnkoutso
Copy link
Contributor

dnkoutso commented Feb 1, 2021

@jrullman its OK I will merge as is.

@dnkoutso dnkoutso merged commit eaa90b6 into CocoaPods:1-10-stable Feb 1, 2021
@jrullman jrullman deleted the add_readme_and_changelog branch February 1, 2021 18:33
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

3 participants