8000 Improve CI for Release by ZUHOWKS · Pull Request #35 · Nephelios/nephelios · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Improve CI for Release #35

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

Merged
merged 7 commits into from
Apr 12, 2025
Merged

Improve CI for Release #35

merged 7 commits into from
Apr 12, 2025

Conversation

ZUHOWKS
Copy link
Collaborator
@ZUHOWKS ZUHOWKS commented Apr 10, 2025

This pull request involves significant updates to the GitHub Actions workflows for CI build and release processes. The changes include the addition of new workflows for CI build and CI release, and the removal of the old CI workflow.

New CI Build Workflow:

  • Added .github/workflows/ci-build.yml to define the CI build process, including steps for checking out the repository, caching dependencies, building the project, running tests, setting up Docker, logging into DockerHub, and building a Docker image.

New CI Release Workflow:

  • Added .github/workflows/ci-release.yml to define the CI release process, triggered by new tags, including steps for checking out the repository, validating the version, setting up Docker, logging into DockerHub, building a Docker image, generating documentation, and deploying to GitHub Pages.

Removal of Old CI Workflow:

  • Removed the outdated CI workflow defined in .github/workflows/ci.yml, which included steps for setting up Rust, installing dependencies, running tests, building documentation, and deploying to GitHub Pages.This pull request includes changes to the GitHub Actions CI workflow configuration to optimize the CI process by ignoring specific file types during push and pull request events.

Changes to .github/workflows/ci.yml:

  • Added paths-ignore to the push event to exclude *.md, .gitignore, .gitmodules, and .env files from triggering the workflow.
  • Added paths-ignore to the pull_request event to exclude *.md, .gitignore, .gitmodules, and .env files from triggering the workflow.

@ZUHOWKS ZUHOWKS self-assigned this Apr 10, 2025
@ZUHOWKS ZUHOWKS requested a review from Copilot April 12, 2025 10:33
Copy link
Contributor
@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

.github/workflows/ci.yml:11

  • The pattern "/.env" may be overly broad, potentially ignoring files like '.env.example'. Consider using a more precise pattern (e.g., "**/.env") if that is the intent.
"**/.env**"

.github/workflows/ci.yml:20

  • The pattern "/.env" may be overly broad, potentially ignoring files like '.env.example'. Consider using a more precise pattern (e.g., "**/.env") if that is the intent.
"**/.env**"

@ZUHOWKS ZUHOWKS force-pushed the fix/ci-path-ignore branch from 9afed33 to 9f53c8e Compare April 12, 2025 14:44
@ZUHOWKS ZUHOWKS changed the title Add ignored path for the ci Improve CI for Release Apr 12, 2025
@ZUHOWKS ZUHOWKS requested a review from Copilot April 12, 2025 15:59
Copy link
Contributor
@Copilot Copilot AI left a 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 3 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • .github/workflows/ci.yml: Language not supported

Copy link
Member
@damien-mathieu1 damien-mathieu1 left a comment

Choose a reason for hiding this comment

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

LGTM juste petite question sur le path ignore

ZUHOWKS and others added 2 commits April 12, 2025 18:34
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ZUHOWKS ZUHOWKS requested a review from Copilot April 12, 2025 16:38
Copy link
Contributor
@Copilot Copilot AI left a 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 3 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • .github/workflows/ci.yml: Language not supported

@ZUHOWKS ZUHOWKS requested a review from damien-mathieu1 April 12, 2025 16:42
@ZUHOWKS ZUHOWKS merged commit 8d44ae0 into main Apr 12, 2025
1 check passed
@ZUHOWKS ZUHOWKS deleted the fix/ci-path-ignore branch April 12, 2025 17:13
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.

2 participants
0