pulsar icon indicating copy to clipboard operation
pulsar copied to clipboard

Uncaught ReferenceError: range is not defined

Open lwlsn opened this issue 10 months ago • 4 comments

Thanks in advance for your bug report!

  • [X] Have you reproduced issue in safe mode?
  • [X] Have you used the debugging guide to try to resolve the issue?
  • [X] Have you checked our FAQs to make sure your question isn't answered there?
  • [X] Have you checked to make sure your issue does not already exist?
  • [X] Have you checked you are on the latest release of Pulsar?

What happened?

Uncaught ReferenceError: range is not defined error is occurring, when running a package (https://github.com/hydra-synth/atom-hydra) that used to function correctly on previous versions of Pulsar

This is the eval-code is keymapped and producing the error

evalCode() { let editor if (editor = atom.workspace.getActiveTextEditor()) { range = { start: { row: 0, column: 0 }, end: { row: editor.getLastScreenRow() + 1, column: 0 } } this.evalFlash(range); this._eval(editor.getText()); } }

Pulsar version

1.115.0

Which OS does this happen on?

🍎 macOS

OS details

Ventura 13.15

Which CPU architecture are you running this on?

Apple M1/M2

What steps are needed to reproduce this?

  1. Install atom-hydra package
  2. Toggle package on
  3. Eval any line of code in pulsar editor

Additional Information:

n/a

lwlsn avatar Apr 04 '24 21:04 lwlsn

Did you follow the instructions in the package's README about installation on Pulsar?

savetheclocktower avatar Apr 05 '24 00:04 savetheclocktower

yes

lwlsn avatar Apr 07 '24 16:04 lwlsn

I can only say that I followed those directions nearly exactly (used npm install instead of ppm install) and the package appeared to work for me. Might be worth trying it again.

Since the maintainer cared enough to update the README to explain how the package can be installed on Pulsar, maybe they'd be willing to publish an updated version to the package repository.

savetheclocktower avatar Apr 07 '24 19:04 savetheclocktower

Sounds like it might be package specific- I've opened up an issue there- thanks!

lwlsn avatar Apr 14 '24 17:04 lwlsn