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

Make samples work with relative path OR with npm module #2

Closed
glennblock opened this issue Dec 13, 2011 · 2 comments
Closed

Make samples work with relative path OR with npm module #2

glennblock opened this issue Dec 13, 2011 · 2 comments

Comments

@glennblock
Copy link
Contributor

A common pattern is for one to copy our samples and just deploy them. Currently you have to modify code as the Azure module is relatively referenced in the require.

A fix would be to add logic that tries to resolve relative, if not found it does a simple require.
@ghost ghost assigned andrerod Dec 13, 2011
andrerod pushed a commit that referenced this issue Jan 26, 2012
Merge pull request #42 from andrerod/dev
ef0946d 
Fixing issue #2 and #33
@andrerod
Copy link

Fixed in dev.

@glennblock
Copy link
Contributor Author

Great!

Sent from my Windows Phone
From: Andre Rodrigues
Sent: 1/26/2012 10:59 AM
To: Glenn Block
Subject: Re: [azure-sdk-for-node] Make samples work with relative path
OR with npm module (#2)
Fixed in dev.

Reply to this email directly or view it on GitHub:
#2 (comment)

andrerod pushed a commit that referenced this issue May 25, 2013
Merge pull request #2 from tistocks/tistocks_HDInsight_2
ab2e6af 
Brought in delete and cleaned up test system.
christav pushed a commit that referenced this issue Apr 3, 2014
Merge pull request #2 from christav/dev
2806305 
Public -> private sync
@jannone jannone mentioned this issue Jun 10, 2014
Closed
@yugangw-msft yugangw-msft mentioned this issue Sep 22, 2014
Merged
huangpf added a commit that referenced this issue Aug 26, 2015
@huangpf
Merge pull request #2 from Azure/master
aa47120 
.
@formulahendry formulahendry mentioned this issue Feb 28, 2018
Closed
@amarzavery amarzavery mentioned this issue May 17, 2018
Closed
AutorestCI added a commit that referenced this issue Nov 28, 2018
@AutorestCI
Generated from 3b62ef631bbcf684da8597627f22d1869bb37c7d
f8123de 
Merge pull request #2 from omarelhariry/kayousef/Dispatch_APIs

Adding dispatch APIs
AutorestCI added a commit that referenced this issue Dec 13, 2018
@AutorestCI
[AutoPR cognitiveservices/data-plane/LUIS/Authoring] Adding azure acc…
ba13503 
…ounts APIs (#4185)

* Generated from 431fadac7b30b7c33b9fa6468874fce839869345

fixing build errors

* Generated from 82cee8e19ad81e9d82d712cfabad1649b91b619a

Merge pull request #1 from omarelhariry/a-moghan/PublishingResponses

Adding Publishing To Regions Status

* Generated from 23a7c077da0b751b8ad2b246d8da041d0906f90d

Fixing formats

* Generated from 4289fc27afed08636bf27abb57ace4a1ff69b7ee

Fixing OAV errors

* Generated from 3b62ef631bbcf684da8597627f22d1869bb37c7d

Merge pull request #2 from omarelhariry/kayousef/Dispatch_APIs

Adding dispatch APIs

* Generated from 4289fc27afed08636bf27abb57ace4a1ff69b7ee

Fixing OAV errors

* Generated from c7d11c54fe4f164073a570f2dba2510ddbf91c76

Add application-json as a 2nd prdocues format for packaging APIs, to describe the error responses

* Generated from b8ee038af90c620395c27635c989a3a6364335a4

Updating operation ids

* Generated from 32e19a3f178eab8d32f3fd79316bbaf7569ca7c8

fix oav and remove error responses
@scbedd scbedd mentioned this issue Jul 20, 2020
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@glennblock @andrerod