Skip to content

Loading…

[Python] Remove rules for bin/, Mr Developer, and Rope #958

Merged
merged 3 commits into from

3 participants

@arcresu

I presume bin/ was initially ignored based on the project layout dictated by some packaging tool, however not all python projects use these layouts. bin/ is normally for executables, so ignoring it by default seems like a bad idea.

As for Mr Developer and Rope: I don't know that they are particularly widespread tools and even if they were, the rules here look suspiciously like those IDE project files that people like to ignore but really shouldn't.

@arcresu

Since nobody has objected to this so far, I'll go ahead and merge. I acknowledge that something still needs to be done to address the use-case of buildout, but I'll defer that to #955.

@arcresu arcresu merged commit 479b7f8 into github:master
@arcresu arcresu deleted the arcresu:python-bin branch
@mfussenegger

@arcresu
What was the reason for removing bin/ ? In projects that use buildout that is a quite common directory to ignore

In addition, this means that VirtualEnv stuff isn't ignored.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 11 deletions.
  1. +1 −11 Python.gitignore
View
12 Python.gitignore
@@ -8,7 +8,6 @@ __pycache__/
# Distribution / packaging
.Python
env/
-bin/
build/
develop-eggs/
dist/
@@ -35,19 +34,10 @@ coverage.xml
# Translations
*.mo
-
-# Mr Developer
-.mr.developer.cfg
-.project
-.pydevproject
-
-# Rope
-.ropeproject
+*.pot
# Django stuff:
*.log
-*.pot
# Sphinx documentation
docs/_build/
-
Something went wrong with that request. Please try again.