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.

Sign up for GitHub

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

Jump to bottom

[CommonRecorder] Custom query-string #14

Open
wants to merge 88 commits into
base: CommonRecorder
Choose a base branch
from
Open

[CommonRecorder] Custom query-string #14

wants to merge 88 commits into from

Conversation

sadasant
Copy link

Fixes Azure#4161

@HarshaNalluru
update rush.json with the new package
b5c0f39
@HarshaNalluru
porting the latest Guidelines from feature/storage
a29370d
@HarshaNalluru
porting the latest guidelines from feature/storage branch
e63ec63
@HarshaNalluru
add readme - test utils - Record and Playback
253679c
@HarshaNalluru
Add disclaimer - mocha and karma
84a1573
@HarshaNalluru
removed the dependency of core-http
5396927
@HarshaNalluru
remove storage utils dependency
4a9269b
@HarshaNalluru
add package.json
404e8f2
@HarshaNalluru
cherry picked commit from feature/storage branch
979c144
@HarshaNalluru
Merge branch 'QureyStringPackage' into CommonRecorder
1bc7c35
@HarshaNalluru
pnpm-lock updated
b727686
@HarshaNalluru
simplify recorder.ts with utils.ts
e4ffd2e
@HarshaNalluru
removed dotenv
b8958f3
@HarshaNalluru
Improved delay module
e6b3f10
@HarshaNalluru
remove delay from utils.ts file
0639d01
@HarshaNalluru
keep only the final recorder module in recorder.ts
387cfd8
@HarshaNalluru
separate customConsoleLog.ts
2c74e8a
@HarshaNalluru
decouple base recorder from recorder
662c70e
@HarshaNalluru
add some return types
8836ea6
@HarshaNalluru
resolve merge conflicts with master
6c92d5d
@HarshaNalluru
update skip list
ec1c239
@HarshaNalluru
ignore readme checks
577c6b0
@HarshaNalluru
add karma.conf.js file in the template
bd0cc21
@HarshaNalluru
add karma dependencies
1dda9cb
@HarshaNalluru
update rollup for consistency
e942898
@HarshaNalluru
update scripts - package.json
a3883a3
@HarshaNalluru
update node test suite
581a9f2
@HarshaNalluru
update unit-test scripts
876df31
@HarshaNalluru
seperate out node tests and browser tests
560bdcf
@HarshaNalluru
👊 is not allowed in headless chrome browser, So, updated to ❤
b074656
HarshaNalluru and others added 25 commits July 25, 2019 17:36
@HarshaNalluru
resolve conflicts with master
6938eb4
@HarshaNalluru
update PNPM lock
d2c19aa
@HarshaNalluru
commit pnpm lock file
a13e72f
@HarshaNalluru
fix browser issue with recorder package
fde7439
@HarshaNalluru
Updating GuideLines with karma setup
59380d5
@HarshaNalluru
final version of karma settings
ccbe7bf
@HarshaNalluru
if (env.ACCOUNT_NAME) replace acc name only if present
8d8b8a4
@HarshaNalluru
update fs-extra module
c46105c
@HarshaNalluru
add a no-op placeholder test
a5815e4
@HarshaNalluru
merge upstream master
eb0667d
@HarshaNalluru
rollup entry point fix
e35bfd2
@HarshaNalluru
update typings path
754413b
@HarshaNalluru
merge upstream master into CommonRecorder
7458f68
@HarshaNalluru
add TODOs
8426897
@HarshaNalluru
explanation for require("path") instead of import
4d2237c
@HarshaNalluru
moved delay() to utils
65dcf53
@HarshaNalluru
delay moved to utils
c102b3e
@HarshaNalluru
isRecording -> isRecordMode, isPlayingBack -> isPlaybackMode
725baa0
@HarshaNalluru
runtime: "node" | "browsers" - union type
f121be7
@HarshaNalluru
testHierarchy -> testSuiteTitle
bd97933
@HarshaNalluru
runtime -> platform
5c789ae
@HarshaNalluru
importNock constant to importNockStatement
12f0152
@HarshaNalluru
update the escapeRegExp function
809acc7
@HarshaNalluru
Elaborating beforeeach, aftereach details and minor improvements - Gu…
55c2c53 
…idelines
@sadasant
[CommonRecorder] Custom query-string
cd02687 
Fixes Azure#4161
@sadasant sadasant force-pushed the CommonRecorder-query-string-alternative branch from 48588c7 to cd02687 Compare August 15, 2019 15:26
@HarshaNalluru HarshaNalluru force-pushed the CommonRecorder branch 3 times, most recently from e8060ba to 7900b45 Compare August 27, 2019 19:02
@HarshaNalluru
Copy link
Owner

@sadasant Rebase and make a PR to Azure/azure-sdk-for-js

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sadasant @HarshaNalluru