ppm: Update submodule to de6a852bd101e388b61fe67b5

[DeeDeeG]

How to do this kind of update:

When the changes you want are live at the default branch of ppm repo (https://github.com/pulsar-edit/ppm), run the following command in the root of the core repo to update the ppm submodule in the core repo:

git submodule update --remote ppm.

Brief Summary of Changes

Includes a lot of decaf work from multiple contributors, a dependency bump, a small code refactor, a Windows postinstall script fix, and switching to our fork of npm 6 that includes node-gyp 9.x.

Also: Skip fetching updates for packages with repository https://github.com/pulsar-edit/pulsar, since these should be bundled packages, which we don't publish updates for on the package registry anymore, and for which we wish to save the API backend some traffic. (I forgot to mention this in the commit message bumping the ppm submodule, dang.)

Includes the following PR's from ppm repo:

• Decaffeinate spec helper (updated) ppm#63
• Decaffeinate apm cli spec ppm#64
• Decaffeinate ci spec ppm#65
• Decaffeinate clean spec ppm#66
• Decaffeinate command spec ppm#67
• Decaffeinate config spec ppm#68
• Decaffeinate develop spec ppm#69
• Decaffeinate disable spec ppm#70
• Decaffeinate docs spec ppm#71
• Decaffeinate enable spec ppm#72
• Decaffeinate featured spec ppm#73
• Decaffeinate help spec ppm#74
• Decaffeinate init spec ppm#75
• Decaffeinate install spec ppm#76
• Decaffeinate link spec ppm#77
• Decaffeinate remaining spec files from list-spec on ppm#78
• Switch to our npm fork, to get newer node-gyp (node-gyp 9.x) ppm#79
• [ Note: PR 80 was already included in the previous ppm bump for this core repo ]
• Move Spec Decaf PRs into master ppm#81
• Make postinstall scripts work on Windows with spaces in cwd path ppm#83
• Decaf Source ppm#84
• Cleanup visualStudioIsInstalled() ppm#85
• Migrate to second-mate and remove first-mate ppm#86
• Update unpublishing wording ppm#90
• Skip update checks for bundled packages ppm#91

[confused-Techie]

Assuming all tests pass, lets get this merged!

[DeeDeeG]

My hope is to let tests run, especially I am looking for the editor tests and settings-view package tests still being green/passing.

I'm also gonna grab a binary off the "Build Pulsar Binaries" CI job and see how it behaves.

[DeeDeeG]

So, we're seeing an issue I know a bit about.

/Users/runner/work/pulsar/pulsar/dist/mac/Pulsar.app: invalid destination for symbolic link in bundle

A change (that I authored 😬) in node-gyp (nodejs/node-gyp@b9ddcd5) leaves some symlinks that are meant to help node-gyp execute the right version of Python. Apple's signing process really doesn't like symlinks pointing to outside the signed binary's files, though. And it errors out with the above message.

There is a fix in node-gyp master already. It was merged last month, not in a tagged or published-to-npm-registry release yet. (nodejs/node-gyp@0f1f667)

There's a few other changes since the latest node-gyp release (v9.4.0), such as "Sync deps and engines with npm" is kinda big, but we are already out of sync with not much overlap between new node-gyp 9's deps and old npm 6's deps. The PR I'm referring to: https://github.com/nodejs/node-gyp/pull/2770.

We either need a different version of node-gyp in our forked npm, or back away from the node-gyp bump in ppm for a bit to get other ppm changes into core.

I suppose ideally I will update the references in pulsar's fork of npm 6 to point to a specific commit of node-gyp repo. But that will take some time. If I can't get to that right away, then I suggest we revert the "npm fork / bump node-gyp" stuff in ppm repo and do that over with something that won't break signing Pulsar editor on macOS.

[DeeDeeG]

PR for ppm that hopefully can address the macOS signing situation, and thereby un-block bumping ppm in Pulsar core repo: pulsar-edit/ppm#94

[confused-Techie]

With #725 open should we go ahead and close this one?

[DeeDeeG]

@confused-Techie I have the other PR set to auto-close this one, so that should take care of it. If not, this can be manually closed. 👍