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

allow changing opts on project settings file (*.sublime-project) #340

Merged
merged 1 commit into from Aug 20, 2021
Merged

allow changing opts on project settings file (*.sublime-project) #340

merged 1 commit into from Aug 20, 2021

Conversation

glaucocustodio
Copy link
Contributor

Allow changing opts (as ctags.opts) on project settings.

Use case: I am adding this as I have a particular project with many files, I want to ignore some of them when generating ctags to speed up things, so I have added the following to the project settings file (ex: myproject.sublime-project):

{
  "settings": {
    "ctags.opts": ["--exclude=@.ctags_ignore"]
  }
}

then I created the file .ctags_ignore containing the folders I want the ctags command to ignore:

node_modules
public/packs

I can't simply add opts to Sublime Text/Packages/User/CTags.sublime-settings as I would need to create a .ctags_ignore file for each project, which I don't want to.

References:

@stephenfin stephenfin merged commit 1ceef7f into SublimeText:development Aug 20, 2021
@stephenfin
Copy link
Collaborator

Thanks! I'll try get this in a release before the end of the weekend

@glaucocustodio
Copy link
Contributor Author

Did you have time to release it @stephenfin?

Changing of topic.. What do you think about passing by default the flag "--exclude=@.ctags_ignore" to the ctags command if a file .ctags_ignore exists at the project root folder?

@glaucocustodio
Copy link
Contributor Author

Would you cut a release please @stephenfin? What do you think about passing by default the flag "--exclude=@.ctags_ignore" to the ctags command if a file .ctags_ignore exists at the project root folder?

@stephenfin
Copy link
Collaborator

Would you cut a release please @stephenfin?

Sorry for the delay: I was on holiday. This is done now.

What do you think about passing by default the flag "--exclude=@.ctags_ignore" to the ctags command if a file .ctags_ignore exists at the project root folder?

If you can draft a PR includes docs explaining the impact this would have, I'll be happy to review this. TBH though, I haven't used this extension in years so I'm reluctant to make any significant changes here before keeping things working since I have no way to test them myself.

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.

None yet

2 participants