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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

馃悰 Handle GitHub repos with redirects #876

Merged
merged 1 commit into from
Aug 22, 2021
Merged

馃悰 Handle GitHub repos with redirects #876

merged 1 commit into from
Aug 22, 2021

Conversation

azeemshaikh38
Copy link
Contributor

  • Please check if the PR fulfills these requirements
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Some repos can redirect to different owner/name. Eg:
  • github.com/chromium/ct-policy -> github.com/GoogleChrome/CertificateTransparency
  • github.com/flavioislima/HeroicGamesLauncher -> github.com/Heroic-Games-Launcher/HeroicGamesLauncher
  • github.com/farag2/Windows-10-Sophia-Script -> github.com/farag2/Sophia-Script-for-Windows
  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
    No.

@azeemshaikh38 azeemshaikh38 enabled auto-merge (squash) August 21, 2021 20:57
@github-actions
Copy link

Integration tests success for 321df017d03fe19e29f2bd82dc28936033cc55ae

@azeemshaikh38 azeemshaikh38 merged commit b7c0d03 into main Aug 22, 2021
inferno-chromium
inferno-chromium approved these changes Aug 22, 2021
View reviewed changes
Copy link
Contributor

@inferno-chromium inferno-chromium left a comment

Choose a reason for hiding this comment

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

Thanks!

@azeemshaikh38 azeemshaikh38 deleted the azeems/redirects branch August 22, 2021 03:45
oliverchang
oliverchang reviewed Aug 22, 2021
View reviewed changes
Copy link
Contributor

@oliverchang oliverchang left a comment

Choose a reason for hiding this comment

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

nice, LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oliverchang oliverchang oliverchang left review comments

@inferno-chromium inferno-chromium inferno-chromium approved these changes

@naveensrinivasan naveensrinivasan Awaiting requested review from naveensrinivasan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@azeemshaikh38 @oliverchang @inferno-chromium @azeemsgoogle