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

Merge masterkey module and component #6385

Merged
merged 4 commits into from
Sep 30, 2021

Conversation

drew2a
Copy link
Contributor

@drew2a drew2a commented Sep 28, 2021

This PR contains location changes of files related to the masterkey component.

The new structure: https://github.com/drew2a/tribler/tree/feature/merge_master_key/src/tribler-core/tribler_core/components/masterkey

image

Addresses #6335

@drew2a drew2a force-pushed the feature/merge_master_key branch 2 times, most recently from 1b76a47 to 0275608 Compare September 29, 2021 09:49
@drew2a drew2a marked this pull request as ready for review September 29, 2021 11:47
@drew2a drew2a requested review from a team and xoriole and removed request for a team September 29, 2021 11:47
ichorid
ichorid previously approved these changes Sep 29, 2021
xoriole
xoriole previously approved these changes Sep 30, 2021
@drew2a drew2a dismissed stale reviews from xoriole and ichorid via 77eba4f September 30, 2021 09:45
@drew2a drew2a force-pushed the feature/merge_master_key branch from 4621f31 to 77eba4f Compare September 30, 2021 09:45
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@drew2a
Copy link
Contributor Author

drew2a commented Sep 30, 2021

retest this please

@drew2a drew2a merged commit a86aaee into Tribler:main Sep 30, 2021
@drew2a drew2a deleted the feature/merge_master_key branch September 30, 2021 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants