-
Notifications
You must be signed in to change notification settings - Fork 64
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 api extractor #68
Conversation
@@ -54,7 +54,7 @@ export const isInCall = (callState: CallState): boolean => !!(callState !== 'Non | |||
/** | |||
* Check if a given string is a GUID. | |||
* Regex expression from: https://stackoverflow.com/a/13653180 | |||
* @param s string to check | |||
* @param s - string to check | |||
*/ | |||
export const isGUID = (s: string): boolean => |
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.
since this is just used for groupId? should we rename s to groupId?
I don't know if calling this "s" is a good idea
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.
the function itself is agnostic to whether its a groupId, it just checks if any string is a guid. I could rename to str
or stringToCheck
?
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.
Awesome, thanks James!
packages/acs-calling-declarative/etc/acs-calling-declarative.api.md
Outdated
Show resolved
Hide resolved
…cation-ui-sdk into jaburnsi/add-api-extractor
What
Add api-extractor
Why
Future option
Once we hit release we could use the api extractor to block checks in any breaking change was made (doesn't support this natively but should be very implementable)
How Tested
View change failing CI: https://github.com/Azure/communication-ui-sdk/pull/68/checks?check_run_id=2114177689