stingray icon indicating copy to clipboard operation
stingray copied to clipboard

Improve stingray.simulator docs

Open Sauravroy34 opened this issue 8 months ago • 2 comments

Fixed for TODO

Sauravroy34 avatar Apr 14 '25 19:04 Sauravroy34

Hi @Sauravroy34 , thanks for the PR, I like how you documented the function. Please run black on the code so that the test passes, it's almost certainly because of a line longer than 100 characters.

matteobachetti avatar Apr 22 '25 13:04 matteobachetti

Hi @Sauravroy34 , thanks for the PR, I like how you documented the function. Please run black on the code so that the test passes, it's almost certainly because of a line longer than 100 characters.

sure , I forgot about that one

Sauravroy34 avatar Apr 22 '25 14:04 Sauravroy34

@Sauravroy34 please rebase, this should fix some failing tests

matteobachetti avatar Jun 25 '25 10:06 matteobachetti

@Sauravroy34 for some reason the Squash and Merge button does not appear. Would you mind squashing all commits into one? Sorry for the inconvenience

matteobachetti avatar Jun 26 '25 06:06 matteobachetti

@Sauravroy34: to make a proper rebase and squash, I suggest the following:

git fetch upstream
git rebase -i upstream/main 

  • In the editor that appears, mark all commits but the first with "s" or "squash"; save

  • In the other editor that appears, edit the single commit message, making it meaningful; save

Finally:

git push --force-with-lease

matteobachetti avatar Jun 26 '25 11:06 matteobachetti

@Sauravroy34: to make a proper rebase and squash, I suggest the following:

git fetch upstream
git rebase -i upstream/main 
  • In the editor that appears, mark all commits but the first with "s" or "squash"; save
  • In the other editor that appears, edit the single commit message, making it meaningful; save

Finally:

git push --force-with-lease

Thanks for helping me out !, can you please verify if I squashed it correctly ?

Sauravroy34 avatar Jun 26 '25 15:06 Sauravroy34

@Sauravroy34 you didn't accept any of my suggested changes, and after the rebase there are more commits than before 😅

matteobachetti avatar Jun 27 '25 06:06 matteobachetti

@Sauravroy34 you didn't accept any of my suggested changes, and after the rebase there are more commits than before 😅

sorry for any inconvenience I have caused shall I close this one and open another pull request ?

Sauravroy34 avatar Jun 27 '25 09:06 Sauravroy34

@Sauravroy34 it might be quicker, yes. No worries, and thanks for the contribution!

matteobachetti avatar Jun 27 '25 09:06 matteobachetti