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

Closed
wants to merge 4 commits into from
Closed

Remove /bin from Eclipse IDE. #4611

wants to merge 4 commits 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?

@Yousha Yousha requested a review from a team as a code owner May 10, 2025 08:33
@wirecat
Copy link
Collaborator

wirecat commented May 23, 2025

Haven't seen any dialogue from the author after giving feedback. Closing for now.

@wirecat wirecat closed this May 23, 2025
@Yousha
Copy link
Contributor Author

Yousha commented May 24, 2025

@wirecat What message? I couldn't find any message...

@wirecat
Copy link
Collaborator

wirecat commented May 29, 2025

@Yousha I was referring to #4611 (comment).

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