Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

List all supported options in README #807

Merged
merged 3 commits into from
May 29, 2020
Merged

List all supported options in README #807

merged 3 commits into from
May 29, 2020

Conversation

sebastianbenz
Copy link
Collaborator

No description provided.

missing tags and extension imports required by a valid AMP document.
Automatically inject any missing markup required by AMP.

* name: `autoAddBoilerplate`
Copy link
Collaborator

Choose a reason for hiding this comment

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

this isn't AMP Boilerplate, right? naming is confusing if I am correct (since you advocate for it being removed laster on)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yup, but changing this would be a breaking API change :-(. Anyway, renamed it to autoAddMandatoryTags and kept the old name as well.

@sebastianbenz sebastianbenz changed the title update optimizer readme List all supported options in README May 29, 2020
@sebastianbenz sebastianbenz merged commit 96881b9 into master May 29, 2020
@sebastianbenz sebastianbenz deleted the optimizer-docs2 branch May 29, 2020 08:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants