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

More symbols #1

Open
lunaroyster opened this issue Sep 21, 2019 · 13 comments
Open

More symbols #1

lunaroyster opened this issue Sep 21, 2019 · 13 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@lunaroyster
Copy link
Owner

lunaroyster commented Sep 21, 2019

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

@lunaroyster lunaroyster added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Sep 21, 2019
@isa3bel
Copy link
Contributor

isa3bel commented Sep 21, 2019

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!

@lunaroyster
Copy link
Owner Author

Thanks @isa3bel!

To run the project, take a look at this: https://github.com/lunaroyster/LaTeX-search/blob/master/react-readme.md

@isa3bel
Copy link
Contributor

isa3bel commented Sep 21, 2019

I think I created a PR for a branch called tax. Was it removed?

@lunaroyster
Copy link
Owner Author

It looked good, so I merged your PR. Here: #3

Again, thanks for contributing!

@isa3bel
Copy link
Contributor

isa3bel commented Sep 22, 2019

are there a list of symbols that still need to be added? I can add some more and create a PR

@isa3bel
Copy link
Contributor

isa3bel commented Sep 22, 2019

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

@lunaroyster
Copy link
Owner Author

lunaroyster commented Sep 22, 2019

@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

https://help.github.com/en/articles/getting-changes-from-a-remote-repository#fetching-changes-from-a-remote-repository

Edit: I'd also recommend working with a GUI client like desktop.github.com or gitkraken.com

@isa3bel
Copy link
Contributor

isa3bel commented Sep 23, 2019

git fetch https://github.com/lunaroyster/LaTeX-search master

I used this command but my local was still not updated.

@lunaroyster
Copy link
Owner Author

try git pull https://github.com/lunaroyster/LaTeX-search master.

fetch only downloads the changes, while pull also merges them

@isa3bel
Copy link
Contributor

isa3bel commented Sep 24, 2019

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.

@lunaroyster
Copy link
Owner Author

It looks like you merged commits from the remote, and pushed to your fork.

@isa3bel
Copy link
Contributor

isa3bel commented Oct 5, 2019

I get these strange errors if I try to run the program on my local machine:

react-scripts start

sh: react-scripts: command not found
npm ERR! code ELIFECYCLE
npm ERR! syscall spawn
npm ERR! file sh
npm ERR! errno ENOENT
npm ERR! latexsearch@0.1.0 start: react-scripts start
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the latexsearch@0.1.0 start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

npm ERR! A complete log of this run can be found in:
npm ERR! /Users/isabelbolger/.npm/_logs/2019-10-05T03_34_37_550Z-debug.log

do you know what the error could be?

@lunaroyster
Copy link
Owner Author

I think you can try prefixing those commands with npx? That indicates you're referring to a package locally installed in latex-search (rather than globally installed). Lemme know if this works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants