Skip to content

Removes this for compatability with pulsar#59

Open
JonBensi wants to merge 1 commit intospadarian:masterfrom
JonBensi:Patch-For-Pulsar
Open

Removes this for compatability with pulsar#59
JonBensi wants to merge 1 commit intospadarian:masterfrom
JonBensi:Patch-For-Pulsar

Conversation

@JonBensi
Copy link

@JonBensi JonBensi commented Aug 8, 2024

This would address and fix issue #58. Tested on both my work laptop (Mac) and personal device (Ubuntu 20.04).

@falcoso
Copy link

falcoso commented Nov 26, 2024

Description

Hello! Thanks for making a version that gets this package to work with Pulsar!. i've tried copying over your branch into my pulsar packages which does get reid of the problem in issue #58, but now I get that "get_all_docblocks" is read-only.

Steps to Reproduce

  1. Clone the Patch-For-Pulsar branch into the ~/.pulsar/packages directory
  2. Open Pulsar

Versions

Pulsar: 1.122.2024112101 x64
Electron: 12.2.3
OS: Ubuntu 22.04.4
Thrown From: docblock-python package 0.19.1

Stack Trace

Failed to load the docblock-python package

At "get_all_docblocks" is read-only.

Error: "get_all_docblocks" is read-only.
    at linter-docblock-python.js:6:38
    at Object.<anonymous> (linter-docblock-python.js:6:38)
    at Object.<anonymous> (linter-docblock-python.js:192:2)
    at Module._compile (/app.asar/src/native-compile-cache.js:120:30)
    at /app.asar/src/compile-cache.js:252:23)
    at Module.load (internal/modules/cjs/loader.js:935:32)
    at Module._load (internal/modules/cjs/loader.js:776:14)
    at Function.f._load (electron/js2c/asar_bundle.js:5:12913)
    at Function.o._load (electron/js2c/renderer_init.js:33:379)
    at Module.require (internal/modules/cjs/loader.js:959:19)
    at require (/app.asar/src/native-compile-cache.js:67:27)
    at Object.<anonymous> (docblock-python.js:7:1)
    at Object.<anonymous> (docblock-python.js:983:2)
    at Module._compile (/app.asar/src/native-compile-cache.js:120:30)
    at /app.asar/src/compile-cache.js:252:23)
    at Module.load (internal/modules/cjs/loader.js:935:32)
    at Module._load (internal/modules/cjs/loader.js:776:14)
    at Function.f._load (electron/js2c/asar_bundle.js:5:12913)
    at Function.o._load (electron/js2c/renderer_init.js:33:379)
    at Module.require (internal/modules/cjs/loader.js:959:19)
    at require (/app.asar/src/native-compile-cache.js:67:27)
    at requireModule (/app.asar/src/module-utils.js:3:17)
    at Package.requireMainModule (/app.asar/src/package.js:916:27)
    at /app.asar/src/package.js:141:16
    at Package.measure (/app.asar/src/package.js:82:19)
    at Package.load (/app.asar/src/package.js:127:10)
    at PackageManager.loadAvailablePackage (/app.asar/src/package-manager.js:701:10)
    at /app.asar/src/package-manager.js:609:14
    at Config.transact (/app.asar/src/config.js:892:14)
    at PackageManager.loadPackages (/app.asar/src/package-manager.js:607:17)

Commands

Non-Core Packages

atom-ide-markdown-service 2.1.0 
busy-signal 2.0.1 
docblock-python 0.19.1 
file-icons 2.1.47 
highlight-selected 0.17.0 
intentions 2.1.1 
linter 3.4.0 
linter-flake8 2.4.0 
linter-ui-default 3.4.1 
markdown-preview-plus 4.11.3 
minimap 4.40.1 
minimap-highlight-selected 4.6.6 
platformio-ide-terminal 2.10.1 
python-autopep8 0.1.3 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants