Conversation
|
I preferred #12652 because that was a bugfix-only and was up to date with our master. |
|
Well, it turns out that the rule is still disabled after #12652 |
|
@J0WI Done. |
| <target host="mit-amps.mit.edu" /> | ||
| <target host="mit150.mit.edu" /> | ||
| <target host="mitei.mit.edu" /> | ||
| <target host="mitei-members.mit.edu" /> |
|
|
||
| <target host="eecs.mit.edu" /> | ||
| <target host="www.eecs.mit.edu" /> | ||
|
|
| <target host="techtv.mit.edu" /> | ||
| <target host="tll.mit.edu" /> | ||
| <target host="wayf.mit.edu" /> | ||
| <!--target host="webmail.mit.edu" /--> |
There was a problem hiding this comment.
WFM, please readd it and remove it from the comments.
There was a problem hiding this comment.
Looks like www has still some issues, but it can be redirected to web. Please have a look at the duplicates in MIT-mismatches.xml
wesinator
left a comment
There was a problem hiding this comment.
Need to add test for calendars to pass rules test
|
@J0WI I'd like to re-enable the MIT ruleset. What do you think we should do? We can't work with this PR because it's from @ghost now. I ran Should I just make another PR that just fixes these minimal problems? I don't want to do an exhaustive check for this huge site. Also, what do you think about moving the keyserver https://pgp.mit.edu to its own ruleset just for that one thing? It's extra high value and I don't think it should depend on a hundred other MIT domains. |
|
There is a lot of work here I would prefer to keep. |
|
Re-enabling a complicated ruleset really means the whole thing needs to be reviewed, so my suggestion to just fix two things and then re-enable it is not a good approach. I'm not really interested in reviewing an old, complicated, massive ruleset like this. In other words if you want to resubmit it as your own PR or something, go ahead, but it will probably sit there for a long time. I would like to move |
|
I already reviewed parts of this PR, so I'll continue to review your changes until this is ready to merge. |
|
I don't think we should merge commits from people who have deleted their accounts. It's like I wrote in 9f9c465 in #13887 :
So in that case one of us would need to rewrite the old commits under their own name, and the other would have to review it. Either way I personally would need to review all the changes and I don't really feel like doing that. |
* Re-enable MIT.xml (#12002) * Re-run Travis * Update MIT.xml * Update MIT.xml * Update MIT.xml * Update MIT.xml * Update MIT.xml * Update MIT-mismatches.xml * Update MIT.xml * Update MIT.xml * Update MIT-mismatches.xml * Update MIT.xml * Update MIT.xml * Update MIT.xml * Update MIT.xml * Update MIT.xml * Normalize securecookie rule
No description provided.