bfirsh / jsnes

A JavaScript NES emulator.
https://jsnes.org
Apache License 2.0
6.1k stars 823 forks source link

Add save/load state examples to README file #406

Closed ninjadynamics closed 2 years ago

ninjadynamics commented 2 years ago

Hello again,

This is a rather simple pull request, just added load/save state examples to the readme file.

bfirsh commented 2 years ago

Super, thanks @ninjadynamics! Looks like your git repository is in a tangle though -- you have a merge commit and some commits from master in there, and there are merge conflicts.

It might be easiest to start a new branch based off bfirsh:master, then cherry pick the commits you want to keep.

Also: would you mind putting the saving state example at the end of that little block of examples? I think have running frames and controls first makes more sense as a narrative of how it works.

ninjadynamics commented 2 years ago

Hi! Sure, I'll try to do some housekeeping! This is actually my first time collaborating on github, so sorry about that! '^__^

ninjadynamics commented 2 years ago

Alright, I think I have everything in order now. :)

My repository contains only 3 little changes:

bfirsh commented 2 years ago

Wonderful, thanks. Unfortunately it still looks like your branch is broken. You might want to google how to rebase a git branch, or create a new branch and cherry pick.

ninjadynamics commented 2 years ago

Unfortunately, I can't see any conflicts on my side... Should I "refork" JSNES, apply the changes and create a new pull req? 01 02

bfirsh commented 2 years ago

You don't need to refork -- probably best to make a new branch, then cherry pick the commits you need. GitHub docs should be able to help you out here, and I can also do myself when I have a bit more time :)

ninjadynamics commented 2 years ago

I tried rebasing as you suggested. All tests green and GitHub reports no merge conflicts. If it doesn’t work, I don’t know what else to do. 😐