Skip to content
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

Clean up top level dotfiles / build files #51444

Closed
jakebailey opened this issue Nov 8, 2022 · 0 comments · Fixed by #51446
Closed

Clean up top level dotfiles / build files #51444

jakebailey opened this issue Nov 8, 2022 · 0 comments · Fixed by #51446
Assignees
Labels
Infrastructure Issue relates to TypeScript team infrastructure

Comments

@jakebailey
Copy link
Member

#51387 introduced a couple more top level files; .gulp.js and .git-blame-ignore-revs.

I really wish I could at least see the word "TypeScript" at the top of our readme when loading the pages, as silly as that sounds.

  • .eslintignore can be moved into .eslintrc.json.
  • .yarnrc could be removed (this file actually breaks the internal repo when checked out in-tree).
  • CopyrightNotice.txt can be moved into scripts.
  • Dockerfile and .dockerignore are for docker tests only, and could be moved into tests/cases/docker as Dockerfile and Dockerfile.dockerignore. Then, the docker test runner can set DOCKER_BUILDKIT=1 and use docker build -f ./test/cases/docker/Dockerfile with cwd set to the repo root to have things work.

I have a branch with each of these done, but I'm turning my TODO list into proper issues, so, filing.

@jakebailey jakebailey self-assigned this Nov 8, 2022
@DanielRosenwasser DanielRosenwasser added the Infrastructure Issue relates to TypeScript team infrastructure label Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infrastructure Issue relates to TypeScript team infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants