8000 Update to #24 to limit mismatches for org pages by electrofelix · Pull Request #40 · rossjrw/pr-preview-action · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update to #24 8000 to limit mismatches for org pages #40

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

Merged
merged 1 commit into from
Apr 22, 2023

Conversation

electrofelix
Copy link
Contributor
@electrofelix electrofelix commented Apr 22, 2023

When the repository matches the org name, which might be slightly more
common for small OSS projects where the owner wishes to move to using an
org to allow it be managed by other contributors over time, the existing
code will assume this means it is for the org level pages.

Consequently updates to a repo such as vagrant-libvirt/vagrant-libvirt
will have it's preview URL using the org style rather than the
repository style.

This change adjust the matching to limit to only allowing repos of the
name <org>.github.io to be presented with the org style gh-pages
preview URL.

Fixes #39

When the repository matches the org name, which might be slightly more
common for small OSS projects where the owner wishes to move to using an
org to allow it be managed by other contributors over time, the existing
code will assume this means it is for the org level pages.

Consequently updates to a repo such as vagrant-libvirt/vagrant-libvirt
will have it's preview URL using the org style rather than the
repository style.

This change adjust the matching to limit to only allowing repos of the
name `<org>.github.io` to be presented with the org style gh-pages
preview URL.

Fixes: rossjrw#39
@rossjrw rossjrw added the bug Something isn't working label Apr 22, 2023
@rossjrw rossjrw merged commit 699c12b into rossjrw:main Apr 22, 2023
@rossjrw
Copy link
Owner
rossjrw commented Apr 22, 2023

Thanks @electrofelix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Check for org pages repo matches unexpectedly
2 participants
0