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

Add 3.13 folder and update nightly #2192

Merged
merged 3 commits into from
Nov 5, 2024

Conversation

ColeHiggins2
Copy link
Contributor

No description provided.

@ColeHiggins2 ColeHiggins2 self-assigned this Nov 4, 2024
</tbody>
</table>
</div>
<td style='white-space:nowrap'>--puppet-dir-group</td>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ekohl Is this supposed to be separated from the table?

@ekohl ekohl changed the title Add 3.13 folder and udpate nightly Add 3.13 folder and update nightly Nov 5, 2024
@ColeHiggins2 ColeHiggins2 requested a review from ekohl November 5, 2024 16:22
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

Given the difficulty with this, perhaps we should first merge #2194 and rebase on top of it?

<td style='white-space:nowrap'>--puppet-dir-group</td>
<td>Group of the base puppet directory, used when puppet::server is false.</td>
<div id="all-installer-options">
<table class="table table-bordered table-condensed">
Copy link
Member

Choose a reason for hiding this comment

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

This looks odd here. Was the text inserted twice?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just copied the output and replaced what was there... So what we see is what was generated

@ekohl
Copy link
Member

ekohl commented Nov 5, 2024

#2194 was merged, so there's now a merge conflict.

Copy link
Member

Choose a reason for hiding this comment

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

@evgeni this is quoting a lot more than I expected. It's not wrong, but really shows there were many (potential) issues before.

More information about compute resources can be found in the [Compute Resources section](/manuals/{{page.version}}/index.html#5.2ComputeResources) and plugins in the [Plugins section](/manuals/{{page.version}}/index.html#6.Plugins).

<!-- Generated based on `foreman-installer -h | grep enable-` -->
<div>
Copy link
Member

Choose a reason for hiding this comment

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

Please make sure this is equal to nightly

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The script didn't do this... I had to manually create these files to match nightly

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

Thanks! Merge whenever you think it's a good time.

@ColeHiggins2 ColeHiggins2 merged commit bde7e8b into theforeman:gh-pages Nov 5, 2024
2 checks passed
ekohl added a commit to ekohl/theforeman.org that referenced this pull request Dec 6, 2024
Fixes: bde7e8b ("Add 3.13 folder and update nightly (theforeman#2192)")
ekohl added a commit that referenced this pull request Dec 6, 2024
Fixes: bde7e8b ("Add 3.13 folder and update nightly (#2192)")
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants