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

qview-nightly: Update to version 9929965521 #1823

Merged
merged 3 commits into from
Jul 16, 2024

Conversation

brian6932
Copy link
Contributor

I'm not really sure why this manifest wasn't being updated by actions properly, as running checkver.ps1 had 0 issues.

Copy link
Contributor

github-actions bot commented Jul 5, 2024

Your changes do not pass checks.

qview-nightly

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@aliesbelik
Copy link
Contributor

I'm not really sure why this manifest wasn't being updated by actions properly, as running checkver.ps1 had 0 issues.

Looks like it completes with Powershell 7.x but fails with 5.x (although Excavator action uses windows-latest (Windows 2022) runner image which should use 7.x as well):

qview-nightly: couldn't match '\A(\d+) (?\d{4}(?:-\d{2}){2}) (?\d+)\Z' in the output of script

Guess it could happen because of using ToString(...) to format extracted updated_at field value which may also be a String.
Using Get-Date should fix this and works for both the versions:

Get-Date $api.updated_at -Format 'yyyy-MM-dd'
or
(Get-Date $api.updated_at).ToString('yyyy-MM-dd')

@brian6932 brian6932 force-pushed the qview-nightly/add-arm64 branch from 2ddec03 to 48aa8c3 Compare July 15, 2024 15:57
@brian6932
Copy link
Contributor Author

/verify

Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

qview-nightly

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@niheaven niheaven changed the title qview-nightly: Add arm64 support qview-nightly: Update to version 9929965521 Jul 16, 2024
@niheaven niheaven merged commit a1dc5a6 into ScoopInstaller:master Jul 16, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants