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

welcome: Add {{welcome-suboptimal}} and {{welcome-anon-suboptimal}} #1955

Closed
wants to merge 0 commits into from

Conversation

NotAGenious
Copy link

Fixes #1950.

@github-actions github-actions bot added Module: welcome dependencies Pull requests that update a dependency file, likely through dependabot labels Mar 18, 2024
@NovemLinguae NovemLinguae self-requested a review September 6, 2024 07:38
Copy link
Member

@NovemLinguae NovemLinguae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The diff on this one is massive.

https://github.com/wikimedia-gadgets/twinkle/pull/1955/files

Perhaps it's because you're using the master branch of your fork for multiple PRs. Or perhaps it's because you ran a linter or your code editor ran a linter and it changed a ton of lines.

I'd recommend closing and resubmitting this PR, but make sure you don't use the master branch. Create a new branch. Each PR you create should get a new and unique branch in your fork :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file, likely through dependabot Module: welcome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

welcome: add {{Welcome-suboptimal}} and {{Welcome-anon-suboptimal}}
2 participants