Skip to content

Allow Actions to be fetched from non-GitHub source#5469

Merged
landongrindheim merged 1 commit intomainfrom
enable-non-github-dotcom-sources-for-actions
Aug 3, 2022
Merged

Allow Actions to be fetched from non-GitHub source#5469
landongrindheim merged 1 commit intomainfrom
enable-non-github-dotcom-sources-for-actions

Conversation

@landongrindheim
Copy link
Copy Markdown
Contributor

I don't think we realized that we lacked the ability to fetch Actions
from non-GitHub.com sources. With this change, we're now able to fetch
Actions from GitHub Enterprise Server instances.

I don't think we realized that we lacked the ability to fetch Actions
from non-GitHub.com sources. With this change, we're now able to fetch
Actions from GitHub Enterprise Server instances.
@landongrindheim landongrindheim marked this pull request as ready for review August 3, 2022 13:51
@landongrindheim landongrindheim requested a review from a team as a code owner August 3, 2022 13:51
details = string.match(GITHUB_REPO_REFERENCE).named_captures
name = "#{details.fetch('owner')}/#{details.fetch('repo')}"
url = "https://github.com/#{name}"
url = "https://#{source.hostname}/#{name}"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I was wondering if we should also grab the scheme from the source, but looks like it isn't available, and people should really use ssl so 🤷

@landongrindheim landongrindheim merged commit c33771f into main Aug 3, 2022
@landongrindheim landongrindheim deleted the enable-non-github-dotcom-sources-for-actions branch August 3, 2022 20:04
@jurre jurre mentioned this pull request Aug 4, 2022
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