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 homepage #1693

Merged
merged 4 commits into from
Nov 23, 2023
Merged

Add homepage #1693

merged 4 commits into from
Nov 23, 2023

Conversation

dragonandy
Copy link
Contributor

Pull request

Purpose
Added homepage containers.

Approach
Added homepage containers with documents

Open Questions and Pre-Merge TODOs
Check all boxes as they are completed

  • Use github checklists. When solved, check the box and explain the answer.

Learning
https://gethomepage.dev/v0.8.0/installation/docker/

Requirements
Check all boxes as they are completed

@dragonandy dragonandy requested review from a team as code owners November 10, 2023 03:21
@github-actions github-actions bot added docs Automatic label repo Automatic label labels Nov 10, 2023
@dragonandy
Copy link
Contributor Author

Anybody look into it?

@angrycuban13
Copy link
Contributor

Anybody look into it?

You did not fix the issue I highlighted - it will fail the linter.

Copy link
Contributor

Choose a reason for hiding this comment

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

This needs a new line at the end of the file

@angrycuban13 angrycuban13 merged commit bb9683c into GhostWriters:master Nov 23, 2023
11 checks passed
@nemchik
Copy link
Member

nemchik commented Nov 24, 2023

FYI I followed up with #1700 to remove files for unsupported CPU types (DS only supports x86_64 and aarch64), and fix a spelling error.

@dragonandy dragonandy deleted the addhomepage branch November 24, 2023 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Automatic label repo Automatic label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants