Skip to content
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

Directory Structure: We should put .idea folder out of the project folder #2

Closed
qiansen1386 opened this issue Dec 13, 2013 · 4 comments
Assignees
Milestone

Comments

@qiansen1386
Copy link

This folder contains the current open tab on your IDE.
The problem is :

  1. It is not suppose to be a part of the project it is just a IDE helper class.
  2. It will force everybody to open all tabs opened by the last commit person.

My solution is clone this directory under other local directory, then we can open the parent folder instead of directly open the yinCart folder, so that we can put the .idea folder outside the project. Or we can change the PHPStorm configuration, but i don't know how to do that yet.

Plus: I think we'd better put the folder like js, css, image under /www folder. What do you think?

这个文件夹是用来存放你现在正在编辑的窗口信息的。
问题在于:

  1. 这玩意儿压根不是银Cart的一部分啊。
  2. 他会强迫其他人使用所有最后一个提交的人打开的窗口。

我的办法是,要么大家在本地把yinCart的文件夹放到一个父文件夹之内,编辑时打开父文件夹,这样.idea就不会在yinCart里面而是外面了。或者改改IDE的设置,不过偶暂时还不会。

另外:我们最好把 js, css, image放到/www文件夹里面。你们觉得呢?

@ghost ghost assigned qiansen1386 Dec 13, 2013
@qiansen1386
Copy link
Author

比如放成WebRoot/YinCart/basic

@gaolujie1989
Copy link
Contributor

I think we should edit the .gitignore file to ignore the IDE folder when we commit, it will be better.

@qiansen1386
Copy link
Author

Excellent Idea! My knowledge set about git is still stuck in GUI stage,
LOL. Could you help to create the file and commit it, so that we can close
this issue.

2013/12/13 gaolujie1989 notifications@github.com

I think we should edit the .gitignore file to ignore the IDE folder when
we commit, it will be better.


Reply to this email directly or view it on GitHubhttps://github.com//issues/2#issuecomment-30490291
.

@yinhe
Copy link
Member

yinhe commented Dec 13, 2013

k, we have removed .idea directory via 244c46e

@yinhe yinhe closed this as completed Dec 13, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants