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

Merge this repository with https://github.com/rubygems/bundler.github.io #591

Closed
12 tasks done
deivid-rodriguez opened this issue Dec 11, 2021 · 18 comments · Fixed by #789 or #794
Closed
12 tasks done

Merge this repository with https://github.com/rubygems/bundler.github.io #591

deivid-rodriguez opened this issue Dec 11, 2021 · 18 comments · Fixed by #789 or #794

Comments

@deivid-rodriguez
Copy link
Member

deivid-rodriguez commented Dec 11, 2021

This is not a big deal, really, but it feels a bit cumbersome to use two different repositories.

As far as I understand, we could either:

  • Use a branch in this repository to deploy to GitHub Pages, instead of a separate repository.

or

  • Stop deploying to GitHub Pages and use Heroku instead.

The former seems simpler, but the latter seems slightly more desirable to me, because we currently have a permanently red Heroku staging pipeline, just to keep Heroku review apps working, which is very useful. I guess we could fix that actually having a staging site that uses heroku, but at that point we may as well switch everything to Heroku.

Anways, this is not a big deal, but it could be an interesting/fun exercise for someone willing to play around with Heroku and Github Pages.

Summary

TODOs

rubygems/bundler-site

rubygems/bundler.github.io

@hsbt
Copy link
Member

hsbt commented Dec 13, 2021

+1

@tnir
Copy link
Collaborator

tnir commented Jul 25, 2022

@deivid-rodriguez Any update?

+1 for the former as deploying to Heroku looks much slower than the current middleman build (50-60s) as you can see in the review apps (3-4 min in multiple PRs, I guess). The former also provides "deploy logs" to all contributors. The former looks like an easy way enough to me and I finished to make a proof of concept of it last weekend.

@deivid-rodriguez
Copy link
Member Author

No updates @tnir, I never planned to tackle this one myself actually.

So feel free to move this forward. So your plan is to deploy to GitHub Pages within this repo and also keep review apps? I think that would be ideal.

@tnir
Copy link
Collaborator

tnir commented Jul 25, 2022

@deivid-rodriguez Yes, my plan is:

  • to enable GitHub Pages in this repo.
  • to push master branch in https://github.com/rubygems/bundler.github.io to this repo as gh-pages.
  • to set custom domain of GitHub Pages to bundler.io in this repo.
  • to unset custom domain in the old repo.
  • to add deprecation notice on the default branch to README in the old repo.
  • to archive the old repo.

and

  • not to change anyhing on review apps in this repo.

The former tasks can be done with Admin role in these two repositories.

@deivid-rodriguez
Copy link
Member Author

Ok, so from now on deployment will be just bundle exec middleman build, and configure GitHub Pages to deploy the build folder in the master branch, correct?

@tnir
Copy link
Collaborator

tnir commented Jul 25, 2022

While we can do so, I am not a fan of it. Use of the default branch "master" would not be changed, so it would be a source branch. On the other hand, gh-pages branch is to be an artifact branch and does exist only as a source of the static site hosting. These are my plan. What do you think?

@deivid-rodriguez
Copy link
Member Author

Ah, ok, yeah using the gh-pages branch sounds good too, I just misunderstood your second bullet point above.

I think we can start with a PR to migrate the deploy task? I think we only need to change it to push to the gh-pages branch inside this repository, instead of pushing to an external repository (and removing all complexities of doing that), correct?

@tnir
Copy link
Collaborator

tnir commented Jul 25, 2022

Yes, you are right!

@tnir tnir self-assigned this Jul 28, 2022
@tnir
Copy link
Collaborator

tnir commented Jul 28, 2022

  1. Set a custom domain as bundler.io in GitHub Pages at https://github.com/rubygems/bundler-site/settings/pages (currently set as http://blog.rubygems.org/bundler-site/)

The above is the instruction a person with Admin role can do whenever they wants to do from now (since the publishing sources are identical in bunder-site and bundler.github.io repos). @hsbt Can you try it?

Resources

@tnir
Copy link
Collaborator

tnir commented Jul 28, 2022

Prior to the above, as an Org admin, you should make sure if bundler.io exists at `https://github.com/organizations/rubygems/settings/pages and add it if not.

@hsbt
Copy link
Member

hsbt commented Jul 28, 2022

Now, I removed custom domain setting from bundler.github.io and added custom domain as bundler.io to bundler-site.

@tnir
Copy link
Collaborator

tnir commented Jul 29, 2022

Thanks for unsetting the custom domain in the old repo.

Just removing CNAME from rubygems/bundler.github.io repo caused occupying https://blog.rubygems.org/bundler.github.io/ on blog.rubygems.org. @hsbt Could you un-archive it to set None as the publishing source at https://github.com/rubygems/bundler.github.io/settings/pages?

@tnir tnir reopened this Jul 29, 2022
@tnir
Copy link
Collaborator

tnir commented Jul 29, 2022

(optional) Add a commit with the deprecation notice on the default branch to README.

Also I prefer adding a link to this repo to both README.md and About in https://github.com/rubygems/bundler.github.io

@hsbt
Copy link
Member

hsbt commented Jul 29, 2022

rubygems/bundler.github.io

I disabled to publish github pages from rubygems/bundler.github.io and archive it.

@tnir
Copy link
Collaborator

tnir commented Jul 29, 2022

Thank you. All tasks were done, so closing.

@tnir tnir closed this as completed Jul 29, 2022
@deivid-rodriguez
Copy link
Member Author

Thanks both, I wanted to do this for a long time 🎉

@simi
Copy link
Member

simi commented Jul 29, 2022

🥳

@tnir
Copy link
Collaborator

tnir commented Jul 29, 2022

Thank you all.

The problem we were having was just filed at: https://github.com/orgs/community/discussions/27583

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