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

Type more classes #9275

Merged
merged 10 commits into from Mar 14, 2024
Merged

Type more classes #9275

merged 10 commits into from Mar 14, 2024

Conversation

ryanbrandenburg
Copy link
Contributor

This is just sprinkling some Sorbet around the nuget Ruby code where it was easy to determine the parameter and return types.

@ryanbrandenburg ryanbrandenburg requested a review from a team as a code owner March 13, 2024 00:03
@github-actions github-actions bot added the L: dotnet:nuget NuGet packages via nuget or dotnet label Mar 13, 2024
@JamieMagee JamieMagee added the sorbet 🍦 Relates to Sorbet types label Mar 13, 2024
@ryanbrandenburg ryanbrandenburg requested a review from a team as a code owner March 13, 2024 03:20
@JamieMagee
Copy link
Contributor

@ryanbrandenburg I added some more types based on yours to get each file to strict or strong.

@ryanbrandenburg
Copy link
Contributor Author

@ryanbrandenburg I added some more types based on yours to get each file to strict or strong.

Thanks! I just had an awkard amount of time so I figured I'd make it easier for someone else to move things to strict/strong, but I didn't think it'd happen in the same PR lol.

@abdulapopoola abdulapopoola merged commit 702d4fb into main Mar 14, 2024
121 checks passed
@abdulapopoola abdulapopoola deleted the dev/rybrande/NugetType branch March 14, 2024 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: dotnet:nuget NuGet packages via nuget or dotnet sorbet 🍦 Relates to Sorbet types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants