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

Download release files URL #814

Merged
merged 3 commits into from
Jul 4, 2023
Merged

Download release files URL #814

merged 3 commits into from
Jul 4, 2023

Conversation

bjoernricks
Copy link
Contributor

@bjoernricks bjoernricks commented Jul 4, 2023

What

Display the requested URL when downloading release files.

Why

GitHub uses a lot of redirects to get the final URL where the corresponding release file will be downloaded. Before this final URL has been shown in the terminal. Because this URL may be very long and may not contain the original requested file name it was difficult to understand what's actually being downloaded. Instead display the requested URL in the terminal when downloading release files.

Checklist

  • Tests

This will allow to define the URL being displayed in the terminal.
@github-actions
Copy link

github-actions bot commented Jul 4, 2023

Conventional Commits Report

Type Number
Changed 1
Added 1

🚀 Conventional commits found.

@codecov
Copy link

codecov bot commented Jul 4, 2023

Codecov Report

Merging #814 (1c6bf70) into main (af1e483) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 1c6bf70 differs from pull request most recent head 2504aa5. Consider uploading reports for the commit 2504aa5 to get more accurate results

@@           Coverage Diff           @@
##             main     #814   +/-   ##
=======================================
  Coverage   89.45%   89.45%           
=======================================
  Files          82       82           
  Lines        5443     5444    +1     
  Branches      765      765           
=======================================
+ Hits         4869     4870    +1     
  Misses        406      406           
  Partials      168      168           
Impacted Files Coverage Δ
pontos/github/api/release.py 98.48% <100.00%> (ø)
pontos/helper.py 92.56% <100.00%> (+0.05%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bjoernricks bjoernricks marked this pull request as ready for review July 4, 2023 12:04
@bjoernricks bjoernricks requested a review from a team as a code owner July 4, 2023 12:04
@bjoernricks bjoernricks marked this pull request as draft July 4, 2023 12:05
@bjoernricks bjoernricks marked this pull request as ready for review July 4, 2023 12:10
@bjoernricks bjoernricks enabled auto-merge (rebase) July 4, 2023 12:11
Before the URL of the response was used. GitHub uses a lot of redirects.
Therefore the response URL contained the last redirected URL. Displaying
the originally requested URL seems to be much better at least for
release files.
@bjoernricks bjoernricks merged commit e760716 into main Jul 4, 2023
11 checks passed
@bjoernricks bjoernricks deleted the download-release-files-url branch July 4, 2023 12:33
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.

2 participants