Skip to content

Reference sourcemaps from GitHub.#1887

Merged
cramforce merged 1 commit intoampproject:masterfrom
cramforce:sourcemaps-github
Feb 10, 2016
Merged

Reference sourcemaps from GitHub.#1887
cramforce merged 1 commit intoampproject:masterfrom
cramforce:sourcemaps-github

Conversation

@cramforce
Copy link
Copy Markdown
Member

This is better than local, but ideally this would point at the right tag.

@cramforce
Copy link
Copy Markdown
Member Author

@erwinmombay Could we pass the tag as a flag to the compile step instead?

@erwinmombay
Copy link
Copy Markdown
Member

@cramforce definitely doable, is that so we can get to the version folder? w.a.o/ver/v0/${sourcemap}?

@cramforce
Copy link
Copy Markdown
Member Author

Nah, we point at Github just with the tag in the URL to get sources at the right version.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@jridgewell
Copy link
Copy Markdown
Contributor

Why point to raw.github.com instead of the Github source code viewer?

@erwinmombay
Copy link
Copy Markdown
Member

@erwinmombay
Copy link
Copy Markdown
Member

@cramforce so since the future tag is going to be $internalRuntimeVersion couldn't we just use that?

@cramforce
Copy link
Copy Markdown
Member Author

@erwinmombay yep!

@cramforce
Copy link
Copy Markdown
Member Author

@jridgewell Not sure what you mean? We need the raw source.

@erwinmombay
Copy link
Copy Markdown
Member

@cramforce LGTM pending, the tag version change to the path

@jridgewell
Copy link
Copy Markdown
Contributor

Never mind my comment.

This is better than local, but ideally this would point at the right tag.
@cramforce
Copy link
Copy Markdown
Member Author

Arghh, it works with the tag, but we are missing all the build artifacts. The right solution is to embed the source file into the map as a data-uri.

@cramforce
Copy link
Copy Markdown
Member Author

But, it is still pretty nice step forward, so lets roll with it.

cramforce added a commit that referenced this pull request Feb 10, 2016
Reference sourcemaps from GitHub.
@cramforce cramforce merged commit 3c3ee09 into ampproject:master Feb 10, 2016
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.

3 participants