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

rename "twinklefluff.js" to "twinklerollback.js" #1975

Open
NovemLinguae opened this issue Jun 22, 2024 · 3 comments · May be fixed by #1977
Open

rename "twinklefluff.js" to "twinklerollback.js" #1975

NovemLinguae opened this issue Jun 22, 2024 · 3 comments · May be fixed by #1977

Comments

@NovemLinguae
Copy link
Member

NovemLinguae commented Jun 22, 2024

Unlike the other module names, "fluff" is not descriptive of what the module does. The word is meaningless to new devs

Thanks to @nardog for the idea.

@RustyCat-1
Copy link

@NovemLinguae Do you want the file only to be renamed, or should the module also be renamed (from Twinkle.fluff to Twinkle.rollback)?

@NovemLinguae
Copy link
Member Author

I'm thinking the module too please.

@RustyCat-1
Copy link

I believe I have the module renamed most of the way, but I'd like to make sure that both fluff and rollback in Twinkle.disabledModules (on config) make the checkbox checked and I'm not quite sure how to do that. (I thought about adding a condition for checking that specific checkbox, but I don't think that's a good idea.

@NovemLinguae NovemLinguae linked a pull request Oct 16, 2024 that will close this issue
@NovemLinguae NovemLinguae linked a pull request Oct 16, 2024 that will close this issue
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 a pull request may close this issue.

2 participants