Skip to content

Avoids HTTPS/HTTP conflicts, and linking to out-of-date documentation. #35

Merged
vincentfretin merged 2 commits into
masterfrom
links
Apr 16, 2023
Merged

Avoids HTTPS/HTTP conflicts, and linking to out-of-date documentation. #35
vincentfretin merged 2 commits into
masterfrom
links

Conversation

@DougReeder
Copy link
Copy Markdown

Avoids HTTPS/HTTP conflicts, and linking to out-of-date documentation.

Copy link
Copy Markdown
Member

@vincentfretin vincentfretin left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution.
I'm not sure why you have changes in dist/aframe-physics-system.js it's hard to tell. If this is just some comments, can you please remove the dist from the PR so we can just merge the documentation changes? We'll recreate the dist on next release.
For now please revert the --ssl option, it fails on Ubuntu 22.04. It's better to continue on the switch to webpack in the other PR #22.

Comment thread package.json Outdated
Comment thread AmmoDriver.md
@vincentfretin vincentfretin self-requested a review April 16, 2023 16:28
Copy link
Copy Markdown
Member

@vincentfretin vincentfretin left a comment

Choose a reason for hiding this comment

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

Thanks.

@vincentfretin vincentfretin merged commit b4b60da into master Apr 16, 2023
@vincentfretin vincentfretin deleted the links branch April 16, 2023 18:00
@DougReeder
Copy link
Copy Markdown
Author

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