Don't download releases-index.json to resolve major version#560
Don't download releases-index.json to resolve major version#560akoeplinger wants to merge 7 commits intoactions:mainfrom
Conversation
Starting with .NET 5 the minor version is always zero. The earlier releases don't get new versions anymore so we can hardcode the minor version instead of downloading releases-index.json to do the lookup.
|
ping? |
|
@HarithaVattikuti would you mind reviewing this? thanks |
|
ping |
It's insane this repo sees so little activity from the maintainers when just using an agent could close 90% of issues by triage and opening PRs and reviewing open PRs. Can't you escalate internally? |
|
yeah. I'm on vacation for the next few weeks but will see what I can do when I'm back |
|
:O maintainer interaction 5-6 hours ago, closing and re-opening the PR. Any chance to get anything more @HarithaVattikuti ? |
|
Hi @akoeplinger👋, thanks for the contribution! This should fix the Prettier formatting error and ensure the build output is up to date. Please let us know in case of any issues. |
There was a problem hiding this comment.
Pull request overview
This PR simplifies .NET channel resolution for major-only version inputs by avoiding a network call to releases-index.json, based on the assumption that .NET 5+ channels always use a .0 minor and that .NET Core 1–3 no longer change.
Changes:
- Remove the
@actions/http-client-based lookup ofreleases-index.jsonfor resolving major-only version inputs. - Hardcode major-to-channel mappings for .NET Core 1–3 and default to
${major}.0for other numeric majors.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@priya-kinthali done, thanks |
|
@priya-kinthali forgot to commit the dist/ file, fixedml. there also seems to be a new npm audit error that looks unrelated to the PR |
Description:
Starting with .NET 5 the minor version is always zero. The earlier releases don't get new versions anymore so we can hardcode the minor version instead of downloading releases-index.json to do the lookup.
Check list: