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

passing add-in url via dist gulp task using --url parameter #167

Merged
merged 2 commits into from
Oct 6, 2016

Conversation

amrfouad22
Copy link
Contributor

passing add-in url via dist gulp task using --url parameter
examples :
gulp dist --url https://myserver.local/
gulp dist --url https://myaddin.azurewebsites.net/

add gulp-replace to devdependecy tests
@coveralls
Copy link

coveralls commented Jun 8, 2016

Coverage Status

Coverage remained the same at 98.287% when pulling 134d4a7 on ministainer:issue-162 into 8417f79 on OfficeDev:master.

@amrfouad22
Copy link
Contributor Author

Hi @jthake ,
I know you might not have the time , but any feedback will be much appreciated 👍

Amr

@amrfouad22
Copy link
Contributor Author

@beth-panx I see that this repo got more love recently which is great , I have this PR sitting still for around 4 months now .

Amr

@beth-panx
Copy link
Member

beth-panx commented Oct 5, 2016

Hi @ministainer I am working on reviewing pull requests as well as closing issues.. We are currently on the stage of prioritizing some of the works that needs to be done. This is def on my task list :)

@@ -1,6 +1,8 @@
'use script';

var gulp = require('gulp');
var replace=require('gulp-replace');
var argv = require('yargs').argv;
Copy link
Member

Choose a reason for hiding this comment

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

Should this package be added to package.json?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@beth-panx it should let me add it and update tests as well

@coveralls
Copy link

coveralls commented Oct 6, 2016

Coverage Status

Coverage remained the same at 98.287% when pulling 75cf865 on ministainer:issue-162 into 8417f79 on OfficeDev:master.

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.

4 participants