Skip to content

[WIP] Auto generate web-types.json to provide autocomplete for JetBrains IDEs#7905

Closed
vaibhavhrt wants to merge 5 commits intovuetifyjs:nextfrom
vaibhavhrt:master
Closed

[WIP] Auto generate web-types.json to provide autocomplete for JetBrains IDEs#7905
vaibhavhrt wants to merge 5 commits intovuetifyjs:nextfrom
vaibhavhrt:master

Conversation

@vaibhavhrt
Copy link

@vaibhavhrt
Copy link
Author

@KaelWD so the method writeJsonFile converts a given js object to json file. I will just have to create an object in js which matches the schema of web-types.json and use writeJsonFile on it. Correct me if I am wrong.

@vaibhavhrt
Copy link
Author

Also I make a lot of commits, hope you guyz squash and merge to keep your repo's git history clean.

@jacekkarczmarczyk
Copy link
Member

jacekkarczmarczyk commented Jul 22, 2019

Thanks for your effort, it should go to the next branch though, we'll squash it
I see you're passing components object to your method, but maybe (I'm not sure, it's just a thought) it would be easier to process our api.json api.js file instead?

@vaibhavhrt vaibhavhrt changed the base branch from master to next July 22, 2019 07:15
@vaibhavhrt
Copy link
Author

@jacekkarczmarczyk so I need to open PR for the branch next?

@vaibhavhrt
Copy link
Author

It's getting messed up, I will close this, create a new branch on my fork from upstream/next and open a new PR.

@vaibhavhrt vaibhavhrt closed this Jul 22, 2019
@vaibhavhrt
Copy link
Author

I can't find api.json, it's api.js and it's created from components and directives. So why not create my own web-types object from components and directives instead of using api.js.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants