Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
core(emulation): refactor emulation settings & CLI flags #11779
core(emulation): refactor emulation settings & CLI flags #11779
Changes from 58 commits
2663a9c
565ee83
bd19dd8
fac4b46
40e2617
1e97bf8
8f38c6a
2716b92
f413348
ce850ef
c7c2d9c
4a61dbf
36b99a3
2f378ff
ce9b4f8
3e5fe75
bab1c15
f3c6d35
673428e
04f623c
055f99f
2f4bc4c
667368b
c416277
ed863bd
314f9e9
163c1d5
f3d8daf
0640d22
7bb30f4
c7e0f79
d612f15
3200813
79af0a7
40698f5
a2ad362
9a230dd
9242bf7
9019612
4c91515
391a7b0
1a91693
9adf303
55247d9
584277c
dd4acce
702fece
78a75b3
c23a4ea
af2946d
cec1586
4a84840
a84d0b8
39d80f8
d833938
2e8b139
1c3731e
8b27252
3caee82
f31a89d
05917d1
31a7b55
09fb558
77d98c7
7a0e1b9
c4a19c6
58179b0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This definitely looks weird interspersing camelCase and kebab-case. Agreed that
--screen-emulation.disabled
also looks weird, but for non-dot ones--no-emulatedUserAgent
is kind of confusing...There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
purposefully don't want to break these up like
throttling
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
correct. I don't wanna grow --help even more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note that doing it this way means
--no-emulated-user-agent
works (it givesfalse
), but--emulated-user-agent
gives''
--emulated-user-agent false
gives (the string)'false'
.better to use
coerce: coerceOptionalStringBoolean,
likeaudit-mode
andgather-mode
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ya good call.