Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

refactor: Make examples executable #90

Merged
merged 1 commit into from
Aug 7, 2018

Conversation

bripkens
Copy link
Contributor

@bripkens bripkens commented Aug 5, 2018

Right now, the examples cannot be executed without manually linking NPM
modules to the correct locations, i.e. the node_modules/ dir in
project root. This makes it really hard to just try out an example as a
user or as a developer working on opencensus-node.

This PR moves the example to a private module under packages/. This
means that lerna will automatically initialise the necessary symlinks
for developers working on this project. Users on the other hand, could
just run npm install and manually execute the examples.

Right now, the examples cannot be executed without manually linking NPM
modules to the correct locations, i.e. the `node_modules/` dir in
project root. This makes it really hard to just try out an example as a
user or as a developer working on opencensus-node.

This PR moves the example to a *private* module under `packages/`. This
means that lerna will automatically initialise the necessary symlinks
for developers working on this project. Users on the other hand, could
just run `npm install` and manually execute the examples.
Copy link
Contributor

@kjin kjin left a comment

Choose a reason for hiding this comment

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

Great idea, thanks!

@kjin kjin merged commit 703eed0 into census-instrumentation:master Aug 7, 2018
kjin pushed a commit to kjin/opencensus-node that referenced this pull request Aug 24, 2018
Right now, the examples cannot be executed without manually linking NPM
modules to the correct locations, i.e. the `node_modules/` dir in
project root. This makes it really hard to just try out an example as a
user or as a developer working on opencensus-node.

This PR moves the example to a *private* module under `packages/`. This
means that lerna will automatically initialise the necessary symlinks
for developers working on this project. Users on the other hand, could
just run `npm install` and manually execute the examples.
kjin pushed a commit to kjin/opencensus-node that referenced this pull request Aug 24, 2018
Right now, the examples cannot be executed without manually linking NPM
modules to the correct locations, i.e. the `node_modules/` dir in
project root. This makes it really hard to just try out an example as a
user or as a developer working on opencensus-node.

This PR moves the example to a *private* module under `packages/`. This
means that lerna will automatically initialise the necessary symlinks
for developers working on this project. Users on the other hand, could
just run `npm install` and manually execute the examples.
kjin pushed a commit to kjin/opencensus-node that referenced this pull request Aug 24, 2018
Right now, the examples cannot be executed without manually linking NPM
modules to the correct locations, i.e. the `node_modules/` dir in
project root. This makes it really hard to just try out an example as a
user or as a developer working on opencensus-node.

This PR moves the example to a *private* module under `packages/`. This
means that lerna will automatically initialise the necessary symlinks
for developers working on this project. Users on the other hand, could
just run `npm install` and manually execute the examples.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants