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.

Sign up for GitHub

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

Jump to bottom

Removal of unnecessary folders #71

Open
sujaypatil96 opened this issue Apr 21, 2022 · 1 comment
Open

Removal of unnecessary folders #71

sujaypatil96 opened this issue Apr 21, 2022 · 1 comment
Assignees
@turbomam
Labels
maintenance setuptools, poetry, actions etc. related modifications

Comments

@sujaypatil96
Copy link
Contributor

Just a very trivial maintenance issue I wanted to bring up.

It seems some of the folders in this repo can be cleaned up:

  • .idea, .vscode and other editor or IDE specific files should not be pushed upstream
  • assets, bin, config, data, downloads, logs doesn't have any contents at the moment?
    • As a general practice, logs should never be pushed upstream
    • Same for downloads
    • Configuration files and other assets are totally okay
  • I'm assuming pre-poetry has files from the repo before the transition to poetry? Now that we have fully poetrified the repo, is it a good idea to remove this directory?

CC: @cmungall @turbomam
@sujaypatil96 sujaypatil96 self-assigned this Apr 21, 2022
@sujaypatil96 sujaypatil96 added the maintenance setuptools, poetry, actions etc. related modifications label Apr 22, 2022
@turbomam
Copy link
Member

Yeah, I'm responsible for a lot of this. Will work on it next week.

@sujaypatil96 sujaypatil96 removed their assignment May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@turbomam turbomam

Labels
maintenance setuptools, poetry, actions etc. related modifications
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sujaypatil96 @turbomam