Skip to content

Remove /bin from Eclipse IDE. #4611

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Remove /bin from Eclipse IDE. #4611

wants to merge 1 commit into from

Conversation

Yousha
Copy link
Contributor

@Yousha Yousha commented Apr 6, 2025

Reasons for making this change

Many PHP/Composer tool packages include a bin folder for their functionality... This .gitignore entry ensures that these folders are ignored, preventing them from being tracked in version control. As a result, they will not be included in remote repositories.

So I THINK this entry should be removed., if Eclipse IDE used as editor for PHP projects.

Links to documentation supporting these rule changes

Projects/Packages that have bin folder:

https://github.com/squizlabs/PHP_CodeSniffer
https://github.com/overtrue/phplint
https://github.com/laravel/installer
etc...

If this is a new template

Merge and Approval Steps

  • Confirm that you've read the contribution guidelines and ensured your PR aligns
  • Ensure CI is passing
  • Get a review and Approval from one of the maintainers

@Yousha
Copy link
Contributor Author

Yousha commented Apr 6, 2025

Or how about create a new .gitignore file for Eclipse for PHP Developers?

@Mohammed1234-hub
Copy link

@Yousha, thats a good idea i think

@wirecat
Copy link
Collaborator

wirecat commented Apr 25, 2025

@Yousha Something we've done before is comment out the line, but then leave a descriptive message above it.
The message could explain why you would uncomment the line, like in the case of PHP development.

How does that sound?

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

Successfully merging this pull request may close these issues.

3 participants