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.

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

Fix/remove non null assertion #5353

Merged

Conversation

m4thieulavoie
Copy link
Collaborator

Pull Request

πŸ“– Description

This PR removes the non-null assertions in the anchor region storybook file.

🎫 Issues

Fixes #5340

πŸ‘©β€πŸ’» Reviewer Notes

πŸ“‘ Test Plan

βœ… Checklist

General

  • I have included a change request file using $ yarn change
  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

⏭ Next Steps

@m4thieulavoie m4thieulavoie force-pushed the fix/remove-non-null-assertion branch from c0358b4 to 3b2a093 Compare November 2, 2021 00:42
m4thieulavoie and others added 3 commits November 3, 2021 11:45
…chored-region.stories.ts

Co-authored-by: Rob Barber <69695238+robarbms@users.noreply.github.com>
…chored-region.stories.ts

Co-authored-by: Rob Barber <69695238+robarbms@users.noreply.github.com>
@chrisdholt
Copy link
Member

@m4thieulavoie looks like prettier and lint may be failing - can you try and run locally?

@m4thieulavoie
Copy link
Collaborator Author

@m4thieulavoie looks like prettier and lint may be failing - can you try and run locally?

Sure! Will get back at it either today or this weekend, sorry for the delay πŸ‘

@chrisdholt chrisdholt merged commit e4ddc05 into microsoft:master Nov 5, 2021
@m4thieulavoie m4thieulavoie deleted the fix/remove-non-null-assertion branch November 5, 2021 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor Anchored Region stories to remove non-null-assertions
4 participants