-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Pull #2282: Add project icon for IntelliJ IDEA
#2283
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
Conversation
elharo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.idea is in our .gitignore file. Not sure what to do about that, but it needs to be addressed.
|
.Editorconfig project has an issue for this feature to be generic, but it’s taking some time to be ready. Yes IDE specific stuff is not nice, that’s why it’s limited to one uncritical file only. In this case it’s an exception, that all projects have to apply, if they want to shine ✨. It’s possible to add this file and still ignore changes on this.It’s then in the repo, like it was added before the ignore.Sent from my iPhoneOn 5 May 2025, at 12:38, Elliotte Rusty Harold ***@***.***> wrote:
@elharo requested changes on this pull request.
.idea is in our .gitignore file. Not sure what to do about that, but it needs to be addressed.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing in the repo should be ignored. I've seen that before, and it is painful. To add this it's necessary to change the .gitignore file so this on file in .idea is not ignored.
|
done |
|
thx |
|
FYI: @romani @nrmancuso |
Add project icon for IntelliJ IDEAIntelliJ IDEA
|
whats your call on this @gnodet? might be rdy to merge and gain feedback. |
|
awesome, thank you. |
Pull #2282: Add project icon for
IntelliJ IDEAproject iconforIntelliJ IDEAcheckstyle/checkstyle#16813project iconforIntelliJ IDEAcheckstyle/checkstyle#16815