tag: add parameters to {{R from sort name}}#2290
tag: add parameters to {{R from sort name}}#2290User-IAmChaos wants to merge 2 commits intowikimedia-gadgets:masterfrom
Conversation
|
Looks like you got the GitHub side of things correct. "Forking" a repo to make your changes is the way you're supposed to do it and that's what you did. The idea is that you don't have edit access to this repo, but can edit your fork, so the fork is where you store your git branch. Copyright isn't a problem since all the code is open source. You can edit your original post to end in "fixes #2281" to automatically link this PR to that issue. Would recommend you do that. Did you get a chance to load up your patch in localhost, do |
| { tag: 'R from non-neutral name', description: 'redirect from a title that contains a non-neutral, pejorative, controversial, or offensive word, phrase, or name' }, | ||
| { tag: 'R from short name', description: 'redirect from a title that is a shortened form of a person\'s full name, a book title, or other more complete title' }, | ||
| { tag: 'R from sort name', description: 'redirect from the target\'s sort name, such as beginning with their surname rather than given name', restriction: 'insideMainspaceOnly' }, | ||
| { tag: 'R from sort name', description: 'redirect from the target\'s sort name, such as beginning with their surname rather than given name', restriction: 'insideMainspaceOnly' , |
There was a problem hiding this comment.
You have an extra space at the end of this line (the one before the comma) that is causing the "linter" test to fail.
2 participants
I still don't know how to use github and I definitely messed up at least something. I think I ended up copying the entire Twinkle code to my own account? I definitely didnt mean to and I hope I didnt break copyright law or anything. Please check my work. I think this addresses Issue 2281, per my comments on that issue.