chore: maintenance - Githubissues

versotile-org / verso

A web browser that plays old world blues to build new world hope

Apache License 2.0

4.55k stars 155 forks source link

chore: maintenance #179

Closed ByteBaker closed 2 weeks ago

ByteBaker commented 3 weeks ago

clippy delints
add deny(clippy::pedantic) to make lint warnings stricter
make code more idiomatic
improve logic in some places or partial rewrites

ByteBaker commented 3 weeks ago

@pewsheen @wusyong if one of you is available, please review this.

wusyong commented 3 weeks ago

Hey sorry for the late reply. I'm been busy on my work recently. We haven't decided if we should follow any clippy rules yet. And I would like to keep compositor as close to servo's one as possible. This PR might not be the right time to merge yet. But feel free to open more if there's any bug fix.

ByteBaker commented 3 weeks ago

I could then remove deny(clippy::pedantic) from the code. Is there anything else you'd like me to change that would make this useful?