-
Notifications
You must be signed in to change notification settings - Fork 1
docs: update GitHub Actions workflow to support pull requests from 'dev' branch and add Netlify deployment steps #46
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
Conversation
…ev' branch and add Netlify deployment steps
…ference in deployment comment
|
🚀 Deployed on https://46--fastapi-turkiye-docs-preview.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (2)
.github/workflows/docs-preview.yml:6
- The inline comment still only mentions 'main' even though the branch list now includes 'dev'. Please update the comment to reflect both branches.
branches: ["main", "dev"] # Triggers on pull requests targeting the main branch
.github/workflows/docs-preview.yml:37
- [nitpick] Consider using a more descriptive and unique alias (for example, combining branch name and PR number) to avoid potential collisions in deployment identifiers.
NETLIFY_ALIAS: ${{ github.event.number }} # Netlify alias for the deployment
…ts and update accordingly
|
🚀 Deployed on https://46--fastapi-turkiye-docs-preview.netlify.app |
for more information, see https://pre-commit.ci
fix: update NETLIFY_ALIAS to use the first 8 characters of the commit SHA
No description provided.