Skip to content

Conversation

@callmehiphop
Copy link
Contributor

It looks like angular-hightlightjs recently added UMD support which broke our doc examples since we loaded the directive before the underlying library.

This PR changed the loading order to fix the broken examples and I've also added the directive locally to prevent any future conflicts.

CDN version is the tip of master, which introduced a breaking change for
our implementation - moving to local version to avoid that in the future

added angular-highlightjs license file
@callmehiphop callmehiphop added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. docs labels Jul 27, 2015
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 27, 2015
@stephenplusplus
Copy link
Contributor

Nice. Good catch. Thanks!

stephenplusplus added a commit that referenced this pull request Jul 27, 2015
docs: added local version of angular-highlightjs
@stephenplusplus stephenplusplus merged commit 572c9d5 into googleapis:master Jul 27, 2015
@callmehiphop callmehiphop deleted the docs-dep-fix branch July 27, 2015 19:00
@stephenplusplus
Copy link
Contributor

Do you think we should cherry pick this into our gh-pages branch manually so that our docs aren't broken until our Wednesday release?

@callmehiphop
Copy link
Contributor Author

Definitely!

sofisl pushed a commit that referenced this pull request Nov 17, 2022
* chore(main): release 3.4.0

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants