search

samdze / playdate-nim

Nim bindings with extra features for the Playdate SDK
66 stars 3 forks source link

ADD Intellij IDEA config files #57

Closed ninovanhooff closed 8 months ago

ninovanhooff commented 9 months ago

Auto-generated project configuration for IntelliJ IDEA IDE's

samdze commented 9 months ago

Do you think those files are valuable to have by default? I'm not familiar enough with IntelliJ IDEA to say it.

ninovanhooff commented 8 months ago

Yes. These are recommended by Jetbrains to be checked in to version control.

These files are quite vanilla. It is what is generated every time you open the project if the files don't exist yet.

samdze commented 8 months ago

Yes, I mean, is it valuable/worth to have something more to maintain here when these are just standard files generated by the IDE anyways? I'm not sure about that

ninovanhooff commented 8 months ago

It's just the convenience of not having to pick changed files to commit.

I can maintain them, don't expect much to change. But I can .gitignore them as well if you don't feel it's necessary

On Sat, Mar 16, 2024 at 4:26 PM Samuele Zolfanelli @.***> wrote:

Yes, I mean, is it valuable/worth to have something more to maintain here when these are just standard files generated by the IDE anyways? I'm not sure about that

— Reply to this email directly, view it on GitHub https://github.com/samdze/playdate-nim/pull/57#issuecomment-2002020280, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACO7DH6NVOYJIZVLKSNX25LYYRQEFAVCNFSM6AAAAABDYE5RC6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBSGAZDAMRYGA . You are receiving this because you authored the thread.Message ID: @.***>

-- -Nino

samdze commented 8 months ago

A well accepted approach is to have IDE specific configurations ignored by git, see Godot .gitignore I think we should generally conform to that, when it makes sense.