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

v4 - Remove azureedge.net fallback logic and update install scripts #572

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

zaataylor
Copy link
Collaborator

@zaataylor zaataylor commented Jan 2, 2025

In response to dotnet/install-scripts#559, we want to remove the CDN fallback logic added for v4 in #566 and update the install scripts once those changes are completed.

This PR currently:

  • Removes the fallback logic
  • Adds in the update install script

Related issue: dotnet/install-scripts#559

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@zaataylor zaataylor changed the title v4- Remove logic for azureedge.net fallback and update install scripts v4- Remove azureedge.net fallback logic and update install scripts Jan 2, 2025
@zaataylor zaataylor changed the title v4- Remove azureedge.net fallback logic and update install scripts v4 - Remove azureedge.net fallback logic and update install scripts Jan 2, 2025
@richlander
Copy link

In reference to: #570

benwells and others added 6 commits January 29, 2025 21:22
* upgrade cache to 4.0.0

* license update

* failure fix

* e2e failure fix
* Create dependabot.yml

* update latest install scripts
Bumps [actions/publish-immutable-action](https://github.com/actions/publish-immutable-action) from 0.0.3 to 0.0.4.
- [Release notes](https://github.com/actions/publish-immutable-action/releases)
- [Commits](actions/publish-immutable-action@0.0.3...v0.0.4)

---
updated-dependencies:
- dependency-name: actions/publish-immutable-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump @actions/http-client from 2.2.1 to 2.2.3

Bumps [@actions/http-client](https://github.com/actions/toolkit/tree/HEAD/packages/http-client) from 2.2.1 to 2.2.3.
- [Changelog](https://github.com/actions/toolkit/blob/main/packages/http-client/RELEASES.md)
- [Commits](https://github.com/actions/toolkit/commits/HEAD/packages/http-client)

---
updated-dependencies:
- dependency-name: "@actions/http-client"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* fix for the check failures

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Aparna Jyothi <[email protected]>
* Bump undici from 5.28.4 to 5.28.5

Bumps [undici](https://github.com/nodejs/undici) from 5.28.4 to 5.28.5.
- [Release notes](https://github.com/nodejs/undici/releases)
- [Commits](nodejs/undici@v5.28.4...v5.28.5)

---
updated-dependencies:
- dependency-name: undici
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

* fix for the check failures

* fix for licensed check failure

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Aparna Jyothi <[email protected]>
@zaataylor
Copy link
Collaborator Author

The changes from main added above include the update install script, now I'll do the same for the other PRs.

@zaataylor zaataylor marked this pull request as ready for review January 29, 2025 21:41
@zaataylor zaataylor requested a review from a team as a code owner January 29, 2025 21:41
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.

5 participants