Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use ppm as basename in getResourcePath checks #36

Merged
merged 1 commit into from
Nov 15, 2022
Merged

Use ppm as basename in getResourcePath checks #36

merged 1 commit into from
Nov 15, 2022

Conversation

jonian
Copy link
Contributor

@jonian jonian commented Nov 12, 2022

This fixes the issue with core packages not showing in settings view caused by the change of apm install folder to ppm in pulsar-edit/pulsar#125.

Copy link
Member

@confused-Techie confused-Techie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@confused-Techie
Copy link
Member

Although while these changes are great, it occurs to me the decafe is done, and I'm in the process of reviewing. Not sure if you'd be open @jonian in opening another PR when I'm done merging those to apply these changes as well, but it is something I can do if needed. Otherwise I'll ping you when that process is over.

As always thanks for the contribution!

@sertonix
Copy link
Contributor

@confused-Techie It is no problem if this is merged first. I will redecaffeinate some files anyway.

@confused-Techie
Copy link
Member

@confused-Techie It is no problem if this is merged first. I will redecaffeinate some files anyway.

Glad to hear, in that case I'll go ahead and merge this one, thanks!

@confused-Techie confused-Techie merged commit 8a2ce6d into pulsar-edit:master Nov 15, 2022
@sertonix
Copy link
Contributor

This is a bit late but I think this pull request is not correct. I think it has caused this error.

@jonian
Copy link
Contributor Author

jonian commented Nov 15, 2022

@sertonix is that possible with ppm submodule in pulsar being https://github.com/pulsar-edit/ppm/tree/2f184753bcaf71e32c8a977f6b2e1d548f8298bb?

@jonian
Copy link
Contributor Author

jonian commented Nov 15, 2022

Also, I have pulsar installed from AUR, that uses this as a patch and I can install lang-gdscript without any issues.

@sertonix
Copy link
Contributor

Ok, then this might fix the problem. Do you know how to update the submodule?

@jonian
Copy link
Contributor Author

jonian commented Nov 15, 2022

@sertonix @confused-Techie in fact this PR fixes that error. I re-installed the latest binary without applying this fix and the result is:
Screenshot from 2022-11-15 20-54-50

@confused-Techie
Copy link
Member

Glad to hear this PR is fixing the error. To update the submodule thatll have to be done as a PR to pulsar

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.

3 participants