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

refactor(tree): Enable explicit return type rule and fix violations #21066

Merged
merged 15 commits into from
May 15, 2024

Conversation

Josmithr
Copy link
Contributor

@Josmithr Josmithr commented May 13, 2024

In preparation for updating the package to use our recommended base eslint config, this PR enabled the explicit-return-types rule in the package (excluding test code) and fixes violations (while disabling the rule in select cases for functions that intentionally derive their return type from other functions/libraries).

AB#6983

@github-actions github-actions bot added area: dds Issues related to distributed data structures area: dds: tree base: main PRs targeted against main branch labels May 13, 2024
@Josmithr Josmithr marked this pull request as ready for review May 14, 2024 23:31
@Josmithr Josmithr requested a review from a team as a code owner May 14, 2024 23:31
@Josmithr Josmithr enabled auto-merge (squash) May 15, 2024 17:44
@Josmithr Josmithr merged commit bb7a956 into microsoft:main May 15, 2024
30 checks passed
@Josmithr Josmithr deleted the tree/function-return-types branch May 15, 2024 18:44
kekachmar pushed a commit to kekachmar/FluidFramework that referenced this pull request May 21, 2024
…icrosoft#21066)

In preparation for updating the package to use our recommended base
eslint config, this PR enabled the `explicit-return-types` rule in the
package (excluding test code) and fixes violations (while disabling the
rule in select cases for functions that intentionally derive their
return type from other functions/libraries).

AB#6983
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: dds: tree area: dds Issues related to distributed data structures base: main PRs targeted against main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants