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

Fix/cd #71

Merged
merged 3 commits into from
Aug 31, 2023
Merged

Fix/cd #71

merged 3 commits into from
Aug 31, 2023

Conversation

rgomezp
Copy link
Contributor

@rgomezp rgomezp commented Aug 31, 2023

One line summary

Enhanced Git staging and updated GitHub push action to the latest stable version.

Description

This PR introduces two significant changes aimed at streamlining our development workflow and reducing potential issues caused by upstream regressions.

Firstly, we've improved the Git staging options in our documentation. Previously, we only utilized git add ., which limited us to staging new and modified files in the current directory and below. Now, we have incorporated git add -A into our process, enabling the CD process to stage all changes including modifications, deletions, and new files.

Secondly, we have updated the GitHub push action to its latest known stable version (v0.6.0). This step is taken as a proactive measure to prevent potential issues arising from upstream regressions. By locking into a stable version, we aim to add an extra layer of reliability to our CI/CD pipeline.

These changes collectively aim to enhance code manageability and pipeline reliability, setting a more robust foundation for future development.

`git add .` : stages new and modified files in the current directory and below.
`git add -u` : stages modifications and deletions, but not new files.
`git add -A` : stages all changes (modifications, deletions, and new files).
@rgomezp rgomezp requested a review from jkasten2 August 31, 2023 21:20
.github/workflows/cd.yml Show resolved Hide resolved
.github/workflows/cd.yml Outdated Show resolved Hide resolved
After switching to commit all via `git add -A`, the diff should be handled automatically.
@rgomezp rgomezp requested a review from jkasten2 August 31, 2023 21:48
@rgomezp rgomezp merged commit 1545830 into main Aug 31, 2023
@rgomezp rgomezp deleted the fix/cd branch August 31, 2023 21:51
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