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

fix gulp file to check for dev mode #13241

Merged
merged 1 commit into from
Feb 2, 2018
Merged

fix gulp file to check for dev mode #13241

merged 1 commit into from
Feb 2, 2018

Conversation

alabiaga
Copy link
Contributor

@alabiaga alabiaga commented Feb 2, 2018

look for amp.js as opposed to non existent amp-babel.js for build in dev mode.

Fixes/Closes/Related-to #13213

@rsimha rsimha self-requested a review February 2, 2018 21:00
Copy link
Contributor

@rsimha rsimha left a comment

Choose a reason for hiding this comment

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

@alabiaga If you want the original issue to be auto-closed due to this fix, you'll have to write Fixes #<number> in its own line, as that's a special GH keyword. See https://help.github.com/articles/closing-issues-using-keywords/

@rsimha
Copy link
Contributor

rsimha commented Feb 2, 2018

This PR will fix the example pages that are currently trying to fetch from the cdn instead of the local server.

screenshot from 2018-02-02 16-15-15

@rsimha rsimha merged commit 199a100 into ampproject:master Feb 2, 2018
@alabiaga alabiaga deleted the 13213-1 branch February 2, 2018 21:25
protonate pushed a commit to protonate/amphtml that referenced this pull request Feb 26, 2018
RanAbram pushed a commit to RanAbram/amphtml that referenced this pull request Mar 12, 2018
protonate pushed a commit to protonate/amphtml that referenced this pull request Mar 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants