Skip to content

Conversation

@laurentS
Copy link
Contributor

Fixes #68

This PR cleans up the codebase a bit by creating reusable schema objects to represent some patterns found throughout the github API.

This should have no functional impact, but helps a bit with readability by removing a lot of redundant lines :)

@laurentS laurentS requested a review from ericboucher March 23, 2022 10:42
@ericboucher
Copy link
Contributor

@aaronsteers @edgarrmondragon is there no way to configure/bypass the sonar cloud tests? They've been pretty useless so far tbh... And there doesn't seem to be a way to change the duplication threshold. Thoughts?

@ericboucher
Copy link
Contributor

ericboucher commented Apr 12, 2022

@laurentS any changes needed after we merged #93 ?

@laurentS
Copy link
Contributor Author

@laurentS any changes needed after we merged #93 ?

done! I think this is GTG.

@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

@ericboucher ericboucher merged commit 0851e80 into main Apr 18, 2022
@ericboucher ericboucher deleted the refactor-sub-objects branch April 18, 2022 20:32
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.

Create shared schema for reuse

3 participants