Versions icon indicating copy to clipboard operation
Versions copied to clipboard

Added `ols-nightly` package to provide Odin Language Server

Open glektarssza opened this issue 8 months ago • 1 comments

Title is self explanatory.

Added an ols-nightly package which access to nightly builds for the DanielGavin/ols tool.

I would have provided a ols-dev and ols package but the main releases do not appear to have artifacts attached. I am looking into adding those into the aforementioned repository via a pull request.

glektarssza avatar Apr 24 '25 15:04 glektarssza

All changes look good.

Wait for review from human collaborators.

ols-nightly

  • [x] Description
  • [x] License
  • [x] Hashes
  • [x] Checkver
  • [x] Autoupdate

github-actions[bot] avatar Apr 24 '25 15:04 github-actions[bot]

Would it be better to use a version format like 20250928-g4a2a71e?

"version": "20250928-g4a2a71e",
"checkver": {
    "url": "https://github.com/DanielGavin/ols/releases/tag/nightly",
    "regex": "datetime=\"(?<year>[0-9]{4})-(?<month>[0-9]{2})-(?<day>[0-9]{2})(.*\\n)*?.*/commit/(?<commit>[0-9a-f]{7})",
    "replace": "${year}${month}${day}-g${commit}"
},

z-Fng avatar Sep 29 '25 14:09 z-Fng

Would it be better to use a version format like 20250928-g4a2a71e?

"version": "20250928-g4a2a71e",
"checkver": {
    "url": "https://github.com/DanielGavin/ols/releases/tag/nightly",
    "regex": "datetime=\"(?<year>[0-9]{4})-(?<month>[0-9]{2})-(?<day>[0-9]{2})(.*\\n)*?.*/commit/(?<commit>[0-9a-f]{7})",
    "replace": "${year}${month}${day}-g${commit}"
},

Possibly? I'm not opposed to making this change/update though, at this point, I've kind of lost both interest in this change and hope that it will ever get merged/accepted.

If this suggestion will get this PR moving I'll happily make it.

glektarssza avatar Sep 29 '25 15:09 glektarssza

I've kind of lost both interest in this change and hope that it will ever get merged/accepted.

Thank you for your patience, and sorry for the long wait.

If this suggestion will get this PR moving I'll happily make it.

I've made the changes mentioned above and it's ready to be merged.

z-Fng avatar Sep 29 '25 15:09 z-Fng

/verify

z-Fng avatar Sep 29 '25 15:09 z-Fng

All changes look good.

Wait for review from human collaborators.

ols-nightly

  • [x] Lint
  • [x] Description
  • [x] License
  • [x] Hashes
  • [x] Checkver
  • [x] Autoupdate

Check the full log for details.

github-actions[bot] avatar Sep 29 '25 15:09 github-actions[bot]

@z-Fng

Thank you for your patience, and sorry for the long wait.

Hey, we got there in the end! That's what ultimately matters, I think!

I've made the changes mentioned above and it's ready to be merged.

Appreciate it! Glad to see this get merged in finally!

Cheers!

glektarssza avatar Sep 29 '25 18:09 glektarssza