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

chore: use sponge download api instead of hardcoded urls #771

Merged
merged 1 commit into from
Aug 15, 2022

Conversation

derklaro
Copy link
Member

Motivation

Sponge provides a downloads api which allows to query the latest version of a sponge artifact. This is much nicer and removes the need to manually ensure that the download url of the sponge versions are always up-to-date.

Modification

Remove hardcoded sponge download urls from versions.json and replaced them with an install step which fetches the latest version from the sponge downloads api.

Result

Easy ensure that the downloaded version is always the latest one instead of relying on manual updates.

@derklaro derklaro added v: 4.X This pull should be included in the 4.0 release in: node An issue/pull request releated to the node module code t: improvement The pull request improves existing code labels Aug 12, 2022
@derklaro derklaro added this to the 4.0.0-RC1 milestone Aug 12, 2022
@derklaro derklaro self-assigned this Aug 12, 2022
@derklaro derklaro modified the milestones: 4.0.0-RC1, 4.0.0-RC2 Aug 12, 2022
@derklaro derklaro changed the base branch from beta to nightly August 15, 2022 06:31
@derklaro derklaro merged commit a914868 into nightly Aug 15, 2022
@derklaro derklaro deleted the sponge-downloads-api branch August 15, 2022 06:31
derklaro added a commit that referenced this pull request Aug 19, 2022
### Motivation
Sponge provides a downloads api which allows to query the latest version of a sponge artifact. This is much nicer and removes the need to manually ensure that the download url of the sponge versions are always up-to-date.

### Modification
Remove hardcoded sponge download urls from versions.json and replaced them with an install step which fetches the latest version from the sponge downloads api.

### Result
Easy ensure that the downloaded version is always the latest one instead of relying on manual updates.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: node An issue/pull request releated to the node module code t: improvement The pull request improves existing code v: 4.X This pull should be included in the 4.0 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants