Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

The health circle around mother spaceship is not correctly centered #36

Open
ketanhwr opened this issue Dec 15, 2017 · 13 comments · May be fixed by #48
Open

The health circle around mother spaceship is not correctly centered #36

ketanhwr opened this issue Dec 15, 2017 · 13 comments · May be fixed by #48

Comments

@ketanhwr
Copy link
Owner

Can be fixed easily since the coordinates need to be modified by a bit.

@Hn5624
Copy link

Hn5624 commented Dec 16, 2017

Fixed it. Also
There seems to be a lot of duplicate functions on the levels.js.
Can i have permission to clean out some more of the unused functions? Game works fine without them, maybe they are code people were trying to rewrite but never finished.
Once I read and understand some more of the code, I will add some light documentation to them so future users will be able to read it easier.

I committed the change onto the masters branch, and i don't have permission to upload my own branch onto the repository in order to do a pull there. Not sure what I have to do to issue a pull request.

@ketanhwr
Copy link
Owner Author

@ShaileshSridhar2403
Copy link

ShaileshSridhar2403 commented Jan 10, 2018

Has this issue been resolved? The circle still isn't centered correctly in the game.

@ketanhwr
Copy link
Owner Author

@flyer2403 it's still open, so yes.

@Hn5624
Copy link

Hn5624 commented Jan 11, 2018 via email

@ketanhwr
Copy link
Owner Author

@Hn5624 I didn't receive any pull request whatsoever.

@ShaileshSridhar2403
Copy link

@Hn5624 could you please make a pull request?

@Hn5624
Copy link

Hn5624 commented Jan 15, 2018 via email

@p-shannon
Copy link

@Hn5624 Oooooh, Okay, I know what you did. You cloned the repo instead of forking it. When you're contributing to a project you don't own, you need to make a fork of it so you that you have ownership (of a fork). Changes you make in your fork can reflected in the main repo (we'll call this upstream for now on) by making a pull request.

Now, a fix should be easy enough I believe;

  1. Make a new repository that has the same name as this one; leave it empty, it'll be filled in step three.
  2. Change the origin of the repository you worked on to match the one you made; something like git remote set-url origin https://github.com/hn562/reflexio.git
  3. From the repo that you worked on push to origin git push origin; At this point you've populated your fork with the work you've done on the cloned upstream.
  4. Now go to this repo from the web browser and click pull requests; then make a new pull request
  5. Make sure the base says 'ketanhwr/master' and the compare says 'hn5624/master'
  6. Lastly fill out the PR with the information suggested by the contributions guide and wait for it to be merged

Hope you're able to follow these instructions as I imagined you following them in my head, this too is my first contribution to an open source project. I'll be fairly active on github if you have and questions.

@Hn5624
Copy link

Hn5624 commented Jan 16, 2018 via email

@p-shannon
Copy link

Yeah I agree with that. Definitely do yourself a favor and get used to the command line interface from now on. Best of luck in open sourced world!

@dilsheen
Copy link

Is this still up for grabs?

@LtBackup LtBackup linked a pull request Feb 27, 2018 that will close this issue
@sans-clue
Copy link

Is this still up for grabs?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants