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

Move gh-pages into main branch #3868

Open
wants to merge 188 commits into
base: main
Choose a base branch
from
Open

Move gh-pages into main branch #3868

wants to merge 188 commits into from

Conversation

Bellangelo
Copy link
Member

@Bellangelo Bellangelo commented May 31, 2024

Closes #3717

What I have done:

  1. Go to gh-pages and move all files under the docs folder which Github supports.
  2. Go to main branch and run git merge gh-pages --allow-unrelated-histories. This merged the 2 branches even if they had unrelated histories.
  3. Updated scripts and references. There are some references in the spec.js files but I think they are unused.

What we need to do after merge:

  1. Go to Settings>Pages and under the Branch section select the main branch and the docs folder`.
    image
  2. After you update the settings, an action will run to deploy the site again.
  3. We keep the gh-pages for a few weeks, just to be able to quickly change in case of a bug in the process, and then we delete it.

How to review
To quickly skip and verify the updated files, just follow the What I have done section and then you can just git diff to distinguished what files have been moved and what files contain actual changes.

MikeRalphson and others added 30 commits June 14, 2018 10:57
gh-pages; Draft registry content
[gh-pages] Hand-craft API templates for leaner output
Make 3.1 schemas retrievable from their $ids
Signed-off-by: GitHub <[email protected]>

Co-authored-by: webron <[email protected]>
* Added formats to the registry as defined by OpenAPI v3

* Fixed date-time format

* Removed boolean as format is not needed
@Bellangelo Bellangelo requested review from a team as code owners May 31, 2024 00:23
@handrews handrews added Housekeeping script Pull requests that update Bash or JavaScript code labels May 31, 2024
docs/.gitignore Outdated
Comment on lines 3 to 5
node_modules/
scripts/
package-lock.json
Copy link
Member

@handrews handrews Jun 3, 2024

Choose a reason for hiding this comment

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

Should these be here? As far as I can tell only _site/ and deploy/ are moving under docs/, but not the scripts and node environment?

Copy link
Member Author

Choose a reason for hiding this comment

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

These were pre-existed but since there isn't any need for them I have removed them here 83770b9

@handrews
Copy link
Member

handrews commented Jun 3, 2024

Aside from the one comment, this looks good to me. I'm just going to hold off actually approving it until we know who has the permissions to do the actual deployment changes and can be sure that they'll merge and make those changes at the same time. [EDIT: Plus we're still considering just making a separate repo]

@lornajane
Copy link
Contributor

I don't have the permissions needed. I see the source, but not the branch picker.

How can we test this before we do it? Did you try it out on o a public fork, or who can do that? I expected the scripts/ directory (currently only exists in main) to need some changes to get the outputs to build in the target folder (we must currently have some "release" step for this, I'm not sure how that works) - did we check that?

.github/workflows/respec.yaml Outdated Show resolved Hide resolved
.github/workflows/respec.yaml Outdated Show resolved Hide resolved
docs/LICENSE Outdated Show resolved Hide resolved
.github/workflows/respec.yaml Show resolved Hide resolved
@handrews handrews requested a review from a team June 4, 2024 18:59
@Bellangelo
Copy link
Member Author

I don't have the permissions needed. I see the source, but not the branch picker.

How can we test this before we do it? Did you try it out on o a public fork, or who can do that? I expected the scripts/ directory (currently only exists in main) to need some changes to get the outputs to build in the target folder (we must currently have some "release" step for this, I'm not sure how that works) - did we check that?

@lornajane My bad. My fork is the https://github.com/Bellangelo/OpenAPI-Specification which deploys to the https://bellangelo.github.io/OpenAPI-Specification/. I have made some extra changes to the deploy / build scripts based on some finding from @ralfhandl . If you check this action https://github.com/Bellangelo/OpenAPI-Specification/actions/runs/9375129267 you can see that it passes.

The way I have been testing the changes in the site so far is mainly using the fork. The fork copies also some repo settings about the deployment but you have to fork through the Github's UI. If you do this, you will be able to deploy the same way we deploy here but in our own url.

scripts/md2html/build.sh Outdated Show resolved Hide resolved
@Bellangelo Bellangelo requested a review from ralfhandl June 5, 2024 22:32
Copy link
Contributor

@ralfhandl ralfhandl left a comment

Choose a reason for hiding this comment

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

Tested on a fork, respec workflow and github pages work fine

@frankkilcommins
Copy link
Contributor

I've created issue OAI/Arazzo-Specification#207 on the Arazzo repo to change our respec job so that it will work with the new structure.

@frankkilcommins
Copy link
Contributor

Also note that #3900 will need to be incorporated into this PR if it's merged prior to this PR.

@lornajane
Copy link
Contributor

@Bellangelo just wanted to drop you a note to say we haven't forgotten about your excellent work, just a lot of moving parts. We will figure it out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Housekeeping script Pull requests that update Bash or JavaScript code
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Should we-organize our GitHub pages to a directory on main?