Skip to content

[IMPROVEMENT] Corrected the tags file format of ctags in .gitignore#908

Merged
cfsmp3 merged 7 commits intoCCExtractor:masterfrom
Souravirus:saju
Feb 23, 2018
Merged

[IMPROVEMENT] Corrected the tags file format of ctags in .gitignore#908
cfsmp3 merged 7 commits intoCCExtractor:masterfrom
Souravirus:saju

Conversation

@Souravirus
Copy link
Copy Markdown
Contributor

Please prefix your pull request with one of the following: [FEATURE] [FIX] [IMPROVEMENT].

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows (check one):

  • I have never used CCExtractor.
  • I have used CCExtractor just a couple of times.
  • I absolutely love CCExtractor, but have not contributed previously.
  • I am an active contributor to CCExtractor.

I have corrected the tags file format of ctags in .gitignore. I have closed the previous pull request because I have sent some extra commits which I have fixed.

@Souravirus
Copy link
Copy Markdown
Contributor Author

Souravirus commented Jan 21, 2018

Please check this commit because when I used ctags with this project, tags file was generated which was not being ignored by .gitignore. So when I checked the .gitignore, there was a ctags column but the '.tags' things was not hiding any files. Therefore, I deleted that line and added the tags file in the .gitignore. If there are any suggestions, then please suggest I would change them.

Copy link
Copy Markdown
Contributor

@bigharshrag bigharshrag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still have .tags files. I haven't built them in a while so I'm not sure if the naming has changed but I wouldn't recommend removing it to prevent someone else committing the tags accidentally. Since they are tags for you, just add it to the .gitignore file.

@Souravirus
Copy link
Copy Markdown
Contributor Author

@bigharshrag Added the .tags file as you suggested

@cfsmp3 cfsmp3 merged commit 393fbd3 into CCExtractor:master Feb 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants