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

AMP Library upgrade #248

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

marcelovani
Copy link

@marcelovani marcelovani commented Nov 20, 2018

This PR makes the amp library up to date with the latest ampproject/amphtml code

The validator-generated.php file was generated using this fork https://github.com/marcelovani/amphtml/tree/marc-generator

The fork is up to date with ampproject/amphtml

We are waiting for Lullabot/amphtml#7 to be merged first to then issue a pr for the php-validator-generated branch

@karens
Copy link
Member

karens commented Apr 9, 2019

@marcelovani I see you have been working on ways to upgrade this library or create a new one. I'm glad to see you doing this. I was wondering myself about whether the Wordpress plugin could be an opportunity to do this better. I'm curious which path you think has the most promise, trying to resurrect this library or starting over? I've been too busy to help much but am supportive of what you're doing.

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