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

Use file scoped namespaces #377

Merged
merged 1 commit into from
Nov 9, 2021
Merged

Use file scoped namespaces #377

merged 1 commit into from
Nov 9, 2021

Conversation

farodin91
Copy link
Member

Part of #375

@farodin91 farodin91 requested a review from prskr November 9, 2021 10:10
@farodin91
Copy link
Member Author

done with dotnet format style.

@farodin91 farodin91 added this to the 0.9 milestone Nov 9, 2021
@farodin91
Use file scoped namespaces
6da615a 
Signed-off-by: Jan Jansen <[email protected]>
@farodin91 farodin91 force-pushed the use-file-scoped-namespaces branch from 0129753 to 6da615a Compare November 9, 2021 10:19
prskr
prskr approved these changes Nov 9, 2021
View reviewed changes
Copy link
Contributor

@prskr prskr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 best PR ever! 😄

@farodin91 farodin91 merged commit d08cc97 into master Nov 9, 2021
@farodin91 farodin91 deleted the use-file-scoped-namespaces branch November 9, 2021 10:27
@farodin91 farodin91 mentioned this pull request Nov 25, 2021
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@prskr prskr prskr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.9
Development

Successfully merging this pull request may close these issues.
2 participants
@farodin91 @prskr