podlomar / kodim.cz

Web for publishing materials from programming courses
1 stars 5 forks source link

feat: ignore JetBrains project files #70

Closed FilipJirsak closed 1 year ago

FilipJirsak commented 1 year ago

Ignore .idea dir from git (JetBrains IntelliJ Idea, WebStorm etc. project files) and http-test dir (HTTP testing scripts for JetBrains IDEs).

FilipChalupa commented 1 year ago

Nevíš prosím o nějakém dobrém vysvětlení, kde se třeba někdo víc rozepsal, že tohle je dobrý nápad? Osobně si myslím, že lokální setup by se neměl propisovat do projektového gitignore. Soubor .gitignore se tím znepřehledňuje. Obzvlášť pro ty, co používají jiné programy.

FilipChalupa commented 1 year ago

O .idea jsem náhodou třeba já už slyšel, ale co je http-test vůbec netuším. Musím to teď nastudovat, abych kodim pochopil nebo je to nějaká JetBrains věc? A když říkám já, tak myslí všichni.

FilipJirsak commented 1 year ago

http-test nikde nenajdeš, tak jsem to pojmenoval já. Normálně by to patřilo do složky s testy, a ještě by to muselo být rozdělené na jednotkové, integrační, a vedle toho by byla tahle složka.

Jinak se to řeší asi jedině komentářem v .gitignore. Právě že lokální setup pro různá IDE se dává do .gitignore, takže tam pak je třeba celá složka .vscode, podobně .idea. Ona to má Idea udělané tak, že se tam část souborů z .idea dává a ten adresář má vlastní .gitignore, ale nechtěl jsem to tím komplikovat, protože asi nemá v tuto chvíli smysl sdílet konfiguraci IntelliJ Idea. Co vím, tak třeba různé opensource projekty mají běžně v .gitignore vyloučení lokálního setupu pro různá IDE – pro VScode, pro JetBrains, NetBeans, Eclipse…

FilipChalupa commented 1 year ago

Co vím, tak třeba různé opensource projekty mají běžně v .gitignore vyloučení lokálního setupu pro různá IDE – pro VScode, pro JetBrains, NetBeans, Eclipse…

Vím, že se to dělá, ale nikdy mi to nepřišlo jako dobrý nápad. Vysvětloval jsem si to tím, že jen autoři opensource .gitgnore neřeší stejně jako někteří neřeší coding style.

FilipChalupa commented 1 year ago

http-test nikde nenajdeš, tak jsem to pojmenoval já.

Takže to je úplně tvá osobní věc, která nesouvisí ani s repozitářem a ani s aspoň trochu známým IDE?

FilipJirsak commented 1 year ago

Upravil jsem to – okomentoval jsem, co je co, http-test jsem z .gitignore vyhodil. @FilipChalupa Je to takhle lepší?

FilipChalupa commented 1 year ago

Za mě určitě lepší. Díky. :)