Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Host only the three latest release and use authentication #149

Merged
merged 6 commits into from
May 22, 2024
Merged

Conversation

baluchicken
Copy link
Contributor

Description

Because of the GitHub rate limit, we cannot handle releases properly.
The release was normally killed during only the last test phase, where it was too late since the empty artifacts were already published. This tries to address this by using authentication for every curl issued during the artifact creation process, also it fails as fast as possible to avoid overwriting the artifacts.
Since the size of the file, even with authentication, we are hitting rate limits so from now on we will host only the three latest releases.

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

@baluchicken baluchicken self-assigned this May 22, 2024
@baluchicken baluchicken requested a review from a team as a code owner May 22, 2024 09:18
Copy link
Member

@bonifaido bonifaido left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job!

@baluchicken baluchicken merged commit 0d6e237 into site-v2 May 22, 2024
1 check passed
@baluchicken baluchicken deleted the site branch May 22, 2024 09:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants