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.

Sign up for GitHub

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

Jump to bottom

Meta: Fix some TODOs #6872

Merged

fregante merged 2 commits into main from todos

Sep 4, 2023

Member

fregante commented Sep 3, 2023

From #6000 (comment)

What I didn't touch:

TODOs dated in the future
selector-observer TODOs, for the most part
has-selector-related TODOs
TODOs dependent on external factors/libraries


          Fix some TODOs

e62a167

fregante added the meta label


          Lint

0903ab9

fregante commented

View reviewed changes

source/features/hide-newsfeed-noise.css

@@ @@ -9,7 +9,6 @@ published a release @@
               edited a gollum (wiki)
               */
-              /* TODO: Drop old classes, no longer used */

Member Author

fregante Sep 3, 2023

Not hurting anyone. Some classes are still used.

fregante merged commit cc90f0d into main

10 checks passed

fregante deleted the todos branch

September 4, 2023 05:16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment