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

Changed javascript @memberOf to @memberof (lowercase o) #431

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Changed javascript @memberOf to @memberof (lowercase o) #431

wants to merge 1 commit into from

Conversation

blaasvaer
Copy link

Can't seem to do this myself locally anymore, as ST has changed the way packages are installed ... therefore I have to do it this way.

According to the JSDocs @memberof is all lowercase.

@nwinch
Copy link

nwinch commented Feb 19, 2017

Any reason this can't be merged? I need it also.

@blaasvaer
Copy link
Author

Not sure he's giving this much attention ...

@PierrickP
Copy link

Any update ?

@blaasvaer
Copy link
Author

Where are we in the process of getting this merged?

@blaasvaer
Copy link
Author

blaasvaer commented Sep 13, 2017

I've fixed this temporarily: add this repo to Sublime Text (3) and install the package.

Here's how:
In Sublime Text:

  1. Shift+CMD+P and select »Package Control: Add Repository«.
  2. Paste this url »https://github.com/blaasvaer/sublime-jsdocs« into the console that shows at the bottom.

When the package finishes installing, then:

  1. Shift+CMD+P and select »Package Control: Install Package«.
  2. Select »sublime-jsdoc«

This should install my forked version of the repo. You should leave it side by side with DocBlockr, as both options (@memberof and @memberof) will appear when you type @mem ... and won't work if you remove DocBlockr.

And you can then remove this repo when (if) DocBlockr gets updated.

gerardroche added a commit to gerardroche/sublime-doc-blockr that referenced this pull request Jan 29, 2018
* pr:
  Changed javascript @memberof to @memberof (lowercase o)

Closes spadgos#431
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.

4 participants