-
Notifications
You must be signed in to change notification settings - Fork 416
Move examples to their own package #1110
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # roslib-examples | ||
|
|
||
| ## Running the examples | ||
|
|
||
| Run `npm start` to view the examples in your browser. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,38 @@ | ||
| { | ||
| "name": "roslib-examples", | ||
| "version": "2.0.0", | ||
| "description": "Example usage of roslibjs", | ||
| "keywords": [ | ||
| "ROS", | ||
| "ros", | ||
| "roslib", | ||
| "roslibjs", | ||
| "robot" | ||
| ], | ||
| "homepage": "https://github.com/RobotWebTools/roslibjs#readme", | ||
| "bugs": { | ||
| "url": "https://github.com/RobotWebTools/roslibjs/issues" | ||
| }, | ||
| "repository": { | ||
| "type": "git", | ||
| "url": "git+https://github.com/RobotWebTools/roslibjs.git" | ||
| }, | ||
| "license": "BSD-2-Clause", | ||
| "author": "Robot Webtools Team <[email protected]> (https://robotwebtools.github.io)", | ||
| "contributors": [ | ||
| { | ||
| "name": "Ezra Brooks", | ||
| "email": "[email protected]", | ||
| "url": "https://github.com/EzraBrooks" | ||
| } | ||
| ], | ||
| "scripts": { | ||
| "start": "http-server .. -o /roslib-examples/src/index.html" | ||
| }, | ||
| "devDependencies": { | ||
| "http-server": "^14.1.1" | ||
| }, | ||
| "dependencies": { | ||
| "roslib": "^2.0.0" | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. roslib will automatically be symlinked from the other directory because it's in the same workspace, this specification only affects the distributed package (if we distribute this package) |
||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -29,7 +29,6 @@ | |
| "eslint-plugin-jsdoc": "^61.1.12", | ||
| "eslint-plugin-prettier": "^5.5.4", | ||
| "globals": "^16.0.0", | ||
| "http-server": "^14.1.1", | ||
| "jiti": "^2.6.1", | ||
| "jsdom": "^27.0.0", | ||
| "jspm": "^4.2.0", | ||
|
|
@@ -53,7 +52,6 @@ | |
| "ws": "^8.0.0" | ||
| }, | ||
| "directories": { | ||
| "example": "examples", | ||
| "test": "test" | ||
| }, | ||
| "engines": { | ||
|
|
@@ -68,8 +66,7 @@ | |
| "//": "The sed expression for eventemitter3 is because the package metadata is incorrect in EventEmitter3's package.json", | ||
| "generate-importmap": "jspm install --release --provider unpkg && sed -i.bak 's;[email protected]/index.mjs;[email protected]/dist/eventemitter3.esm.js;' importmap.js && rm importmap.js.bak", | ||
| "prepare": "npm run build && npm run generate-importmap", | ||
| "make-changelog": "changelog-maker RobotWebTools roslibjs --markdown | cat - CHANGELOG.md > NEW_CHANGELOG.md && mv NEW_CHANGELOG.md CHANGELOG.md", | ||
| "examples": "http-server . -o /examples/index.html" | ||
| "make-changelog": "changelog-maker RobotWebTools roslibjs --markdown | cat - CHANGELOG.md > NEW_CHANGELOG.md && mv NEW_CHANGELOG.md CHANGELOG.md" | ||
| }, | ||
| "repository": { | ||
| "type": "git", | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This confuses me. Is it
rosliborroslibjs? Where does the difference come from?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The package is actually called
roslib, I guess because historically the "JS" part was redundant because it was the only roslib and it was being published as a javascript package?We can decide to change the name later when we eventually adopt an organization namespace on NPM since that will require changing the name anyway