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

Enable some new test assemblies to be friends #365

Conversation

ChrisBurgessMS
Copy link
Contributor

@ChrisBurgessMS ChrisBurgessMS commented Apr 9, 2024

Testing of Service Fabric requires the ability to reach into this assembly and use internal classes. There are new binaries that need access to this assembly's internals so the assembly needs new entries to make those internal accessible.

# Conflicts:
#	src/Microsoft.ServiceFabric.Actors.KVSToRCMigration/Friend.cs
@ChrisBurgessMS ChrisBurgessMS removed the request for review from yashagarwal23 April 9, 2024 15:31
@pakunapamsft pakunapamsft reopened this Apr 9, 2024
@olegsych
Copy link
Member

olegsych commented Apr 9, 2024

Thanks @ChrisBurgessMS. Could you update the PR description to explain what is being changed and why?

https://cbea.ms/git-commit/#seven-rules

@ChrisBurgessMS ChrisBurgessMS changed the title User/chrisburgessm/release 6.1/microsoft service fabric services friends Enable some new test assemblies to be friends Apr 9, 2024
@ChrisBurgessMS
Copy link
Contributor Author

Thanks @ChrisBurgessMS. Could you update the PR description to explain what is being changed and why?

https://cbea.ms/git-commit/#seven-rules

The what is within the PR. The why has been added to the description.

@olegsych olegsych self-assigned this Apr 9, 2024
@ChrisBurgessMS ChrisBurgessMS merged commit 01c816a into release_6.1 Apr 9, 2024
2 checks passed
@olegsych olegsych deleted the user/chrisburgessm/release_6.1/Microsoft_ServiceFabric_Services_Friends branch June 19, 2024 17:31
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.

3 participants