Skip to content
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

Add webpack-cli to dev dependencies #390

Closed
wants to merge 1 commit into from

Conversation

SeanKilleen
Copy link

@SeanKilleen SeanKilleen commented Dec 11, 2018

Contributes to #389 (I think).

Not sure of the best way to build / test / verify this, but happy to do so with a point in the right direction.

@SeanKilleen
Copy link
Author

Closing in favor of more discussion on #389.

@akrantz
Copy link
Collaborator

akrantz commented Dec 11, 2018

To test it:

  1. In a command window at the generator-office directory, build and link:
c:\generator-office> npm run build
c:\generator-office> npm run link
  1. go to another empty directory where you can create a new project.
    cd c:\test
  2. link to generator-office package
    npm link generator-office
  3. run Yo Office and choose the project template to create.
    yo office
    Choose React, TypeScript, Outlook. Type project name, e.g. "React TypeScript Outlook Add-in".
    cd "React TypeScript Outlook Add-in"
  4. build and run to verify the add-in works.

@akrantz
Copy link
Collaborator

akrantz commented Dec 11, 2018

After you do this, you can do "npm uninstall -g generator-office" and then "npm install -g generator-office" to reinstall the latest released package.

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.

None yet

2 participants