-
Notifications
You must be signed in to change notification settings - Fork 20
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
More symbols #1
Comments
Hi, I added a tax symbol. I am new to open source, so let me know if this was done correctly. I created a PR. Also, how do I test my changes locally on my machine? Thanks! |
Thanks @isa3bel! To run the project, take a look at this: https://github.com/lunaroyster/LaTeX-search/blob/master/react-readme.md |
I think I created a PR for a branch called tax. Was it removed? |
It looked good, so I merged your PR. Here: #3 Again, thanks for contributing! |
are there a list of symbols that still need to be added? I can add some more and create a PR |
also, since you changed master, how do I get the latest on my local machine? if I do git pull it says I have the latest even though I don't since you changed a bit of wording from my PR |
@isa3bel I updated the first comment to include a list. You need to merge the upstream repo. Here are some resources: https://help.github.com/en/articles/merging-an-upstream-repository-into-your-fork Edit: I'd also recommend working with a GUI client like desktop.github.com or gitkraken.com |
git fetch https://github.com/lunaroyster/LaTeX-search master I used this command but my local was still not updated. |
try
|
got it thanks! So I got the latest changes, committed, and then pushed. Did I push to the forked repo rather than directly to your master? I just pushed, but didn't sent out a PR, since that's unnecessary. |
It looks like you merged commits from the remote, and pushed to your fork. |
I get these strange errors if I try to run the program on my local machine:
sh: react-scripts: command not found npm ERR! A complete log of this run can be found in: do you know what the error could be? |
I think you can try prefixing those commands with |
It's simple. Let's add more symbols.
Here's a good place to start: https://oeis.org/wiki/List_of_LaTeX_mathematical_symbols
Here are symbols requests by users: https://airtable.com/shrwk7aGIsiDIDwAe/tblX2jFkbt0J1f88F
I'd begin with symbols in more common use by beginners
The text was updated successfully, but these errors were encountered: