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

create a static version of amp.dev #6472

Merged
merged 7 commits into from
Oct 26, 2022
Merged

create a static version of amp.dev #6472

merged 7 commits into from
Oct 26, 2022

Conversation

patrickkettner
Copy link
Collaborator

No description provided.

@ampproject ampproject deleted a comment from netlify bot Oct 12, 2022
@patrickkettner
Copy link
Collaborator Author

netlify bot hasnt been configured, so their failures are expected

@patrickkettner
Copy link
Collaborator Author

ping @sebastianbenz

Copy link
Collaborator

@sebastianbenz sebastianbenz left a comment

Choose a reason for hiding this comment

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

Did a first path. Overall looks good. Would be good if we could split-up / simplify the build file a bit.

examples/source/1.components/amp-list.html Show resolved Hide resolved
netlify/configs/amp.dev/netlify.toml Outdated Show resolved Hide resolved
gulpfile.js/build.js Show resolved Hide resolved
gulpfile.js/build.js Show resolved Hide resolved
gulpfile.js/build.js Outdated Show resolved Hide resolved
gulpfile.js/build.js Outdated Show resolved Hide resolved
gulpfile.js/build.js Outdated Show resolved Hide resolved
gulpfile.js/build.js Show resolved Hide resolved
gulpfile.js/build.js Outdated Show resolved Hide resolved
gulpfile.js/build.js Outdated Show resolved Hide resolved
with '#' will be ignored, and an empty message aborts the commit.  # #
On branch static-version # Your branch is up to date with
'origin/static-version'.  # # Changes to be committed: # modified:
gulpfile.js/build.js #	modified: netlify/configs/amp.dev/netlify.toml #
.github/workflows/check-grow-link-integrity.yaml # modified:
.github/workflows/lint-js.yaml #	modified:
.github/workflows/test-playground.yaml # # Untracked files: #
.github/workflows/release-static-production.yml #
@sebastianbenz
Copy link
Collaborator

@patrickkettner What's the status of this? From my point of view this looks good, once build.js has been split up.

@patrickkettner
Copy link
Collaborator Author

I am splitting it up. had been working on the github action portion. will ping you on this once its separated

@patrickkettner
Copy link
Collaborator Author

@sebastianbenz build passes

Copy link
Collaborator

@sebastianbenz sebastianbenz left a comment

Choose a reason for hiding this comment

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

thanks! well done!

@patrickkettner patrickkettner merged commit 152b4d9 into future Oct 26, 2022
@patrickkettner patrickkettner deleted the static-version branch October 26, 2022 12:37
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