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

Set up Liam ERD on GitHub Pages #1

Merged
merged 3 commits into from
Dec 26, 2024
Merged

Set up Liam ERD on GitHub Pages #1

merged 3 commits into from
Dec 26, 2024

Conversation

FunamaYukina
Copy link
Member

@FunamaYukina FunamaYukina commented Dec 25, 2024

Summary

I've set up a GitHub Actions workflow and configured the Pages and Environments settings.

Setting
https://github.com/giselles-ai/erd/settings/pages
https://github.com/giselles-ai/erd/settings/environments

GitHub Page
https://giselles-ai.github.io/erd/

ss 2599

Related Issue

Changes

Testing

Action Log
https://github.com/giselles-ai/erd/actions/runs/12492944120/job/34860817326

Other Information

I'm still learning GitHub Actions, so a detailed review would be very helpful🙏.

@FunamaYukina FunamaYukina marked this pull request as ready for review December 25, 2024 12:28
Copy link

@MH4GF MH4GF left a comment

Choose a reason for hiding this comment

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

Thanks! I added some minor comments!

- main
paths:
- .github/workflows/github-pages.yml
- migrations/**/*.sql
Copy link

Choose a reason for hiding this comment

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

This line is not necessary because this file is not in giselles-ai/erd due to repository split.

Suggested change
- migrations/**/*.sql

Copy link
Member Author

Choose a reason for hiding this comment

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

I see.Thank you for teaching me🙏.
I've removed it. c98687f

- name: Prepare the input file (dump.sql)
shell: bash
run: |
pg_dump_docker_image="postgres:16.6-bookworm" # ubuntu-latest pg_dump command is a bit old.
Copy link

Choose a reason for hiding this comment

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

nits: The version of postgres in line 34 is 15, so it should match either.

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, sorry.
I've corrected it.(use postgres:16.6-bookworm ) c98687f

@FunamaYukina
Copy link
Member Author

@hoshinotsuyoshi
It would be helpful if you could give me some feedback.🙏

Copy link
Member

@shige shige left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

Copy link

@hoshinotsuyoshi hoshinotsuyoshi left a comment

Choose a reason for hiding this comment

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

Left two small comments. LGTM! :)

- .github/workflows/github-pages.yml
workflow_dispatch:
schedule:
# Schedule: Runs at 9:00 AM JST every day.

Choose a reason for hiding this comment

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

nits: It's unclear how frequently this schedule will be viewed by Japanese users, so feel free to remove this note.

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, you are right🙏.
I've deleted the note. 28c5373


jobs:
# NOTE: Following the example below, the job is structured into distinct build and deploy phases.
# see https://docs.github.com/ja/pages/getting-started-with-github-pages/using-custom-workflows-with-github-pages

Choose a reason for hiding this comment

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

Copy link
Member

@masutaka masutaka Dec 26, 2024

Choose a reason for hiding this comment

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

It is also good to let the browser settings determine the language. 😄
https://docs.github.com/pages/getting-started-with-github-pages/using-custom-workflows-with-github-pages

Copy link
Member Author

Choose a reason for hiding this comment

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

That's correct as well.👍 Thank you!
28c5373

Copy link
Member

@masutaka masutaka left a comment

Choose a reason for hiding this comment

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

LGTM!

workflow_dispatch:
schedule:
# Schedule: Runs at 9:00 AM JST every day.
- cron: "0 0 * * *"
Copy link
Member

Choose a reason for hiding this comment

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

FYI:

This is not relevant for this repository, but public repositories have this limitation.

🔗 https://docs.github.com/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows#schedule

In a public repository, scheduled workflows are automatically disabled when no repository activity has occurred in 60 days. For information on re-enabling a disabled workflow, see Disabling and enabling a workflow.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you for the information! I didn't know that. 👀

Copy link

@satococoa satococoa left a comment

Choose a reason for hiding this comment

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

🙆‍♂️ 👍

@FunamaYukina
Copy link
Member Author

Thank you all for reviewing.✨

@FunamaYukina FunamaYukina merged commit c8b6c9e into main Dec 26, 2024
2 checks passed
@FunamaYukina FunamaYukina deleted the gh-pages branch December 26, 2024 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants