Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update our "See also" mappings for Clojure 1.9 #2171

Closed
bbatsov opened this issue Jan 20, 2018 · 5 comments
Closed

Update our "See also" mappings for Clojure 1.9 #2171

bbatsov opened this issue Jan 20, 2018 · 5 comments

Comments

@bbatsov
Copy link
Member

bbatsov commented Jan 20, 2018

There are just a simple edn file here. Not sure how exactly @ckoparkar generated it, but I guess it can be updated manually.

@ckoparkar
Copy link
Contributor

ckoparkar commented Jan 20, 2018

The ClojureDocs source repo has a MongoDB database which has a collection called see-alsos. I used that to generate the initial edn file. I re-wrote a script (here) which should extract the data into the current format.

Steps:

  1. Run mongod
  2. Clone https://github.com/zk/clojuredocs and execute bin/db-reset (see local-data)
  3. Run the script, which should generate an edn file

But all of this useless right now because the repository has old data which was last updated 3 years ago. I'll file an issue there and see how it goes. We could always update this manually, but that might be a tedious process.

Note: Why not just use the bson file from the repo ?
From what I could tell, the bson file is not in the correct format. Specifically, it has a map on each line, but the whole thing is not an array.
So it's

{a}
{b}

instead of

[
  {a},
  {b}
]

@ckoparkar
Copy link
Contributor

Done zk/clojuredocs#185

@bbatsov
Copy link
Member Author

bbatsov commented Jan 20, 2018

But all of this useless right now because the repository has old data which was last updated 3 years ago. I'll file an issue there and see how it goes. We could always update this manually, but that might be a tedious process.

I don't think they've added that much to Clojure in those 3 years, but it probably won't be fun to do a manual update. I guess this means that the list doesn't include things from Clojure 1.7 and 1.8 as well.

@ckoparkar
Copy link
Contributor

Oops. You're right. The database was updated on Oct, 2014. So it was already out-of-date when we last used it.

@bbatsov
Copy link
Member Author

bbatsov commented Jan 22, 2018

Fixed by clojure-emacs/cider-nrepl#477

@bbatsov bbatsov closed this as completed Jan 22, 2018
bbatsov added a commit that referenced this issue Jan 22, 2018
The actual update happened in cider-nrepl.
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

2 participants