Skip to content

Remove content for removed features in 2027 #3040

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

Merged
merged 10 commits into from
Jun 25, 2025
Merged

Conversation

Gold856
Copy link
Contributor

@Gold856 Gold856 commented May 27, 2025

References to old Java tools, Labview, and other features removed in 2027 WPILib have been removed.

@Gold856 Gold856 force-pushed the 2027 branch 2 times, most recently from 9decb52 to bcb1a3b Compare May 27, 2025 20:45
@Gold856 Gold856 changed the title [draft] 2027 docs [draft] Remove files for removed features in 2027 May 27, 2025
@Gold856 Gold856 changed the title [draft] Remove files for removed features in 2027 [draft] Remove content for removed features in 2027 May 27, 2025
@sciencewhiz
Copy link
Collaborator

I won't have a chance to review in detail for a while. However, from a usability perspective (and SEO) I think I'd tend to prefer replacing things with stubs that point to replacement rather then a wholesale delete for 2027. Since it's a massive PR, and I've only been able to glance at it, I can't tell if that's what you've done.

See also #3041. I think it might be easier to reorganize first then delete if we're keeping stubs.

@Gold856
Copy link
Contributor Author

Gold856 commented May 28, 2025

It's just wholesale deletes with some minor spot fixes to remove references or use alternate ones, e.g. replacing Shuffleboard with Elastic. And yeah, I think a reorganization first might be easier. Although I'm not exactly sure if stubs would be particularly useful compared to just using rediraffe. I suppose if we use one stub for every major removed component like SmartDashboard and Shuffleboard, that'd be fine.

What I might do is move the spot fixes (the first commit) into a separate PR, which would allow us to be more aggressive in moving/deleting stuff. If you're going to review any part of this, review the first commit only. The rest is just wholesale deletes.

@sciencewhiz
Copy link
Collaborator

Although I'm not exactly sure if stubs would be particularly useful compared to just using rediraffe. I suppose if we use one stub for every major removed component like SmartDashboard and Shuffleboard, that'd be fine.

Right, that was what I was envisioning. Redirect any SmartDashboard article to the stub, and the stub can explain why it's no longer supported and link to alternates. I think that's a better user experience then redirecting any smartdashboard article to some other dashboard article without any explanation.

@Gold856 Gold856 force-pushed the 2027 branch 7 times, most recently from f868d1a to 33d4c92 Compare June 20, 2025 07:38
@Gold856
Copy link
Contributor Author

Gold856 commented Jun 20, 2025

I decided that I do want to do wholesale deletes and redirect to a stub. This mostly comes from the fact that since the reorg will necessitate that articles be moved around anyways, it will require all articles to find a new home, and a lot of articles that are currently up for removal (such as all the Java tools) are a mix of tutorial and reference, so we'd have to spend time to split them up, only to delete them. I think just deleting everything now, with all deletions pointing to one article listing all the removed features, and alternatives, will make things easier in the future.

@Gold856 Gold856 force-pushed the 2027 branch 2 times, most recently from d606628 to b5ce626 Compare June 20, 2025 08:23
@Gold856 Gold856 force-pushed the 2027 branch 5 times, most recently from a543d3c to 503e279 Compare June 21, 2025 17:07
@sciencewhiz
Copy link
Collaborator

files deleted should be removed from the ignore list in .github/workflows/inspector.json

@sciencewhiz sciencewhiz requested a review from arbessette June 21, 2025 22:13
@Gold856 Gold856 changed the title [draft] Remove content for removed features in 2027 Remove content for removed features in 2027 Jun 22, 2025
@sciencewhiz sciencewhiz merged commit 4b8feca into wpilibsuite:2027 Jun 25, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants