-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Remove xrefs from modules #41954
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
Comments
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
/remove-lifecycle stale |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
/remove-lifecycle stale |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
/remove-lifecycle stale |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
/lifecycle frozen |
/remove-lifecycle stale |
Is this still a valid ticket, @bergerhoffer? |
@peterroth yes it is still valid, but unfortunately is not a quick/easy fix. They don't need to just be removed, they will each need to be evaluated and see if we need to add them in as a workaround elsewhere. I think we likely have a few hundred :/ |
I see, @bergerhoffer. In this case I can't fix the ticket because I don't have the reliable resource to build and test the changes. |
Agreed. This one will likely need to be done by our team, and likely broken up for several writers to do. There are likely other tickets that are more suited for anyone to pick up, if you're looking to contribute more. Let me know if you need any help finding something, if you are! |
In reviewing a module, I noticed that there was an xref in it. A quick search of the modules/ directory showed quite a few in there.
We currently don't allow xrefs in modules per our guidelines. We should do a sweep and remove these xrefs. (Comment out or otherwise leave a note so that they can be found/added back in later, if/when we do allow them again).
The text was updated successfully, but these errors were encountered: