-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
(chore): Using PowerShell Core instead of PowerShell Desktop #46
Conversation
Is there any specific reason to stick to PowerShell Desktop instead of PowerShell Core? Will anything break if we switch from PowerShell Desktop to PowerShell Core? If something breaks and we can fix it on the Scoop side, we can work on it. But, if it needs to be fixed on the PowerShell Core side - we can try to work on it. Most of my manifests leverage on checkver.script field and these scripts are developed under PowerShell Core. So, these are not compatible with PowerShell Desktop. That's why, I want to switch from PowerShell Desktop to PowerShell Core in my Excavator GitHub Actions. If we want to stick on PowerShell Desktop for some reason, I have to migrate the scripts to PowerShell Desktop compatibility. Since PowerShell Core is the future, I think that migration is a waste of resources. Anyway, If migration is necessary - I am ready for it. |
Scoop should work out of the box. Even the Scoop Core project itself makes limited use of the new features available in PowerShell Core. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it introduces issues, we can always revert it back.
It's GitHub Actions, using PowerShell should be ok, IMO. |
Hi, sorry for late repsonse on this, but this has to be reverted as it is intentional to use There had been issues of invalid JSON manfiests being committed to official buckets, led to checks for Scoop being taken down on repology. We don't want this to happen again and that's why For 3rd party buckets, if you don't care about PowerShell 5 compatiblity, which is still officially supported by Scoop, you're always free to fork this actions repo and switch to the fork with the running shell changed to Tip: if you are not sure why one line of code is written like that or want to know the commit histroy of that line of code, you may use git blame to do such investigation, which may provide you context you might not know, before taking further action. It is also available on the GitHub web interface. |
Reverts by 944bf8e and requires further investigation |
…ScoopInstaller#46)" This reverts commit 944bf8e.
No description provided.