Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

chore: Adding missing variable in array of dependency in React hook #4130

Merged
a-b-r-o-w-n merged 2 commits into from
Sep 15, 2020
Merged

chore: Adding missing variable in array of dependency in React hook #4130

a-b-r-o-w-n merged 2 commits into from
Sep 15, 2020

Conversation

@LouisEugeneMSFT
Copy link
Contributor

@LouisEugeneMSFT LouisEugeneMSFT commented Sep 14, 2020

#minor

@coveralls
Copy link

coveralls commented Sep 15, 2020

Coverage Status

Coverage remained the same at 55.5% when pulling 246dfd8 on leugene/minorIntellisenseFix into 673f961 on main.

@a-b-r-o-w-n a-b-r-o-w-n merged commit 70f12ea into main Sep 15, 2020
@a-b-r-o-w-n a-b-r-o-w-n deleted the leugene/minorIntellisenseFix branch September 15, 2020 00:15
@cwhitten cwhitten mentioned this pull request Nov 13, 2020
Merged
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
@LouisEugeneMSFT
chore: Adding missing variable in array of dependency in React hook (m…
d4ac211 
…icrosoft#4130)

* stringifying object examples

* minor fix

Co-authored-by: Louis Eugene <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@a-b-r-o-w-n a-b-r-o-w-n Awaiting requested review from a-b-r-o-w-n

@beyackle beyackle Awaiting requested review from beyackle

@boydc2014 boydc2014 Awaiting requested review from boydc2014

@cwhitten cwhitten Awaiting requested review from cwhitten

@GeoffCoxMSFT GeoffCoxMSFT Awaiting requested review from GeoffCoxMSFT

@hatpick hatpick Awaiting requested review from hatpick

@srinaath srinaath Awaiting requested review from srinaath

1 more reviewer

@tonyanziano tonyanziano tonyanziano approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@LouisEugeneMSFT @coveralls @tonyanziano @a-b-r-o-w-n