Skip to content

Remove doc folder (old archived spec and assets), word2md script#51791

Merged
jakebailey merged 1 commit intomicrosoft:mainfrom
jakebailey:remove-doc
Jan 5, 2023
Merged

Remove doc folder (old archived spec and assets), word2md script#51791
jakebailey merged 1 commit intomicrosoft:mainfrom
jakebailey:remove-doc

Conversation

@jakebailey
Copy link
Copy Markdown
Member

Removed in this PR:

  • The archived spec, in doc/PDF formats and associated images.
  • The build scripts / task that convert from the word doc to markdown. I have no idea if this actually works post-mjs conversion.
  • An old redirect for the handbook, which has long since been moved to the website.
  • An old redirect for the wiki, which these days is right in the navbar of the repo.
  • logo.svg, which appears to be unreferenced.

The only thing I'm actually worried about is logo.svg as maybe this is the canonical source for this file.

If we want these back, they're in the git history forever, so we can grab them if needed (and link to release-4.9 if really required).

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Dec 6, 2022
Copy link
Copy Markdown
Member

@ahejlsberg ahejlsberg left a comment

Choose a reason for hiding this comment

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

Feeling a bit nostalgic seeing this go, but as long as there is some way to find the files again, I'm fine with it.

BTW, looks like our branding assets are kept here, so not too worried about logo.svg.

@jakebailey
Copy link
Copy Markdown
Member Author

Funnily, logo.svg is used in one place... webpack's readme. Guess they'll have to move that somewhere else. logo.svg here is barely a logo, it's just the text "TypeScript".

@fatcerberus
Copy link
Copy Markdown

An old redirect for the wiki, which these days is right in the navbar of the repo.

TIL there's a "Wiki" button in the navbar. 😄

@sandersn
Copy link
Copy Markdown
Member

@jakebailey do you still need Daniel's review to merge this?

@jakebailey
Copy link
Copy Markdown
Member Author

I'd prefer to have consensus that I'm not hurting anything/anyone by doing this; I have the deletion itch and this was one I sent as it removes a bunch of build code and a whole top-level directory.

@DanielRosenwasser
Copy link
Copy Markdown
Member

I feel the same as Anders, but ultimately we can remove it.

@yuhuung
Copy link
Copy Markdown

yuhuung commented Jul 25, 2023

May I ask why these files like doc/spec-ARCHIVED.md are removed?

@jakebailey
Copy link
Copy Markdown
Member Author

The spec was inaccurate, being asked about (even though it was inaccurate), and also meant we had to have and maintain some extra goofy build tasks to convert it from a word doc (!) to markdown.

@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

8 participants