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

surf: Open Diff to custom construction #139

Merged
merged 1 commit into from
Aug 25, 2023
Merged

surf: Open Diff to custom construction #139

merged 1 commit into from
Aug 25, 2023

Conversation

slack-coder
Copy link
Contributor

Put the Diff type in lines with its associated types by opening it up to construction by crate consumers. The existing associated types have all their fields public.

Open construction up by making the Diff::insert_* methods public and updating the Diff's stats manually. Diff::stat correctness is covered by existing tests.

Put the Diff type in lines with its associated types by opening it up to
construction by crate consumers.  The existing associated types have all
their fields public.

Open construction up by making the `Diff::insert_*` methods public and
updating the Diff's stats manually.  `Diff::stat` correctness is covered
by existing tests.
@slack-coder
Copy link
Contributor Author

It looks like the linter is overly pedantic on the code base about changes unrelated to this PR.

@FintanH
Copy link
Collaborator

FintanH commented Aug 25, 2023

It looks like the linter is overly pedantic on the code base about changes unrelated to this PR.

Hmmm weird that it just showed up.

@FintanH
Copy link
Collaborator

FintanH commented Aug 25, 2023

It looks like the linter is overly pedantic on the code base about changes unrelated to this PR.

Hmmm weird that it just showed up.

It's also not showing up for me locally. Even more curious 🤔

@FintanH
Copy link
Collaborator

FintanH commented Aug 25, 2023

It looks like the linter is overly pedantic on the code base about changes unrelated to this PR.

Hmmm weird that it just showed up.

It's also not showing up for me locally. Even more curious 🤔

Ah, the toolchain went to 1.72.0, I'll merge this and also push a fix for that

@FintanH
Copy link
Collaborator

FintanH commented Aug 25, 2023

It looks like the linter is overly pedantic on the code base about changes unrelated to this PR.

Hmmm weird that it just showed up.

It's also not showing up for me locally. Even more curious 🤔

Ah, the toolchain went to 1.72.0, I'll merge this and also push a fix for that

Seems like it's a false positive so I've reported an issue here rust-lang/rust-clippy#11402

@FintanH FintanH merged commit 60e919a into radicle-dev:main Aug 25, 2023
11 of 12 checks passed
@slack-coder slack-coder deleted the diff-pub branch August 25, 2023 13:45
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.

2 participants