Skip to content

Support for React #36

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

Closed
achtan opened this issue Nov 24, 2016 · 13 comments
Closed

Support for React #36

achtan opened this issue Nov 24, 2016 · 13 comments

Comments

@achtan
Copy link

achtan commented Nov 24, 2016

pls add support for React className

@huygn
Copy link

huygn commented Jan 7, 2017

+1 would be awesome

@Jacobfred88
Copy link

+1

@JanneHarju
Copy link

+1

can someone else make this if author is occupied.

@JanneHarju
Copy link

I just wrote support for javascript (.js -files) and for react typescript files (.tsx) but I think that I don't have permission to push new branch to repository.

@zignd
Copy link
Owner

zignd commented Apr 13, 2017

Hi @JanneHarju, I'm the repository owner, you can create a pull request with your changes, I'd be glad to accept it. In case you're not familiar with the concept of pull requests, GitHub has some documents explaining how you would go in order to create one.

@JanneHarju
Copy link

@zignd I try to make pull request with source tree put 403 error occur. I make own branch for my changes. When I make pull request it says that it try to push those changes at same time.

@zignd
Copy link
Owner

zignd commented Apr 13, 2017

@JanneHarju Consider using the GitHub website to create the pull request, it's easier for that. Remember that you need to fork the project, then you make the changes to the forked repository and after that, you make a pull request to merge the forked repository into this repository.

@JanneHarju
Copy link

Yes I make branch and trying to push that but I don't have permission to make branch I think. Here is what sourde tree says.
git -c diff.mnemonicprefix=false -c core.quotepath=false push -v --tags --set-upstream origin Support_for_react:master
Pushing to ssh://git@github.com/Zignd/HTML-CSS-Class-Completion.git
ERROR: Permission to Zignd/HTML-CSS-Class-Completion.git denied to JanneHarju.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

@zignd
Copy link
Owner

zignd commented Apr 13, 2017 via email

@JanneHarju
Copy link

OK now I managed to fork it and make pull request. How soon you think you can merge and publish it to market?

@JanneHarju
Copy link

Now I have problem with debugginf extension. debugging Visual studio appears and soon just shutdown. Do you have same problem?

@zignd
Copy link
Owner

zignd commented Apr 14, 2017

OK now I managed to fork it and make pull request. How soon you think you can merge and publish it to market?

@JanneHarju Great, I'm going to start reviewing it right now and if everything goes fine I will be publishing it to the market today.

Now I have problem with debugginf extension. debugging Visual studio appears and soon just shutdown. Do you have same problem?

@JanneHarju It usually works for me, that's weird.

@zignd zignd changed the title FR: support for React Support for React Apr 14, 2017
@zignd
Copy link
Owner

zignd commented Apr 14, 2017

Feature added in the latest version thanks to @JanneHarju.

@zignd zignd closed this as completed Apr 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants