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 root of repo and release archives #4934

Closed
dom96 opened this Issue Oct 22, 2016 · 6 comments

Comments

Projects
None yet
3 participants
@dom96
Copy link
Member

dom96 commented Oct 22, 2016

There is far too many files in the root of the repo. IMO the following files should be removed/moved:

  • bootstrap.sh
  • build_tools.sh
  • contributing.txt
  • docstyle.rst
  • finish.nim
  • install.txt
  • install_nimble.nims
  • install_tools.nims
  • readme.txt
  • start.bat

Similarly I will need to look at the release tarballs/zip, I'm sure there is room for improvement there too.

@Araq

This comment has been minimized.

Copy link
Member

Araq commented Jul 1, 2018

Should be closed once install_tools.nims and install_nimble.nims have been removed.

@Araq Araq added Easy Cleanup and removed Installation labels Jul 1, 2018

@dom96 dom96 added the Hacktoberfest label Oct 2, 2018

dom96 added a commit that referenced this issue Oct 3, 2018

Merge pull request #9152 from barbeque/4934-remove-old-install-scripts
Remove install_nimble.nims, install_tools.nims as per #4934
@dom96

This comment has been minimized.

Copy link
Member

dom96 commented Oct 3, 2018

readme.txt and install.txt should still be removed IMO

@Araq

This comment has been minimized.

Copy link
Member

Araq commented Oct 3, 2018

They can be merged into a single readme.txt but the txt extension is quite important for Windows.

@dom96

This comment has been minimized.

Copy link
Member

dom96 commented Oct 4, 2018

I disagree. Every Windows user who's a developer knows how to open a .md file. It's not hard.

@Araq

This comment has been minimized.

Copy link
Member

Araq commented Oct 9, 2018

Fine, whatever.

@Utwo

This comment has been minimized.

Copy link
Contributor

Utwo commented Oct 26, 2018

So should install.txt be merged in readme.md?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment