[#506] Update contributing guidelines to reflect latest assumptions #524
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 tasks
placek
force-pushed
the
chore/506-update-contributing-guidelines-to-reflect-latest-assumptions
branch
from
239297e
to
862068e
Compare
MSzalowski
approved these changes
Mar 25, 2024
The changes made include updating the CONTRIBUTING.md document to include moving the ticket to the 'in review' status when the PR is ready for review, changing the state of the PR from draft to actual PR, addressing all change requests before merging the branch into `develop`, and automating the deployment process to the test and staging environments. These modifications aim to streamline the development workflow by ensuring that tickets, PRs, and branches are managed efficiently during the development and review phases.
placek
force-pushed
the
chore/506-update-contributing-guidelines-to-reflect-latest-assumptions
branch
from
862068e
to
58afed1
Compare
placek
deleted the
chore/506-update-contributing-guidelines-to-reflect-latest-assumptions
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #506.
This pull request aims to update the project's contributing guidelines to reflect recent changes in the development workflow. The modifications include moving the ticket to the 'in review' status when the PR is ready for review, transitioning the PR from draft to an actual PR, addressing all change requests before merging into the
developbranch, and automating the deployment process to the test and staging environments. These updates are intended to enhance the management of tickets, PRs, and branches, streamlining the development process and ensuring more efficient handling during the development and review phases.The changes in this pull request involve updating the CONTRIBUTING.md document to incorporate the latest development workflow practices. The recommendation is made against using rebase and merge, suggesting the use of proper merges for easier feature cherry-picking. Additionally, details are added to explain the benefits of proper merges over rebase and merge, clarifying how they facilitate cherry-picking features. A new section is introduced to explain the purpose and usage of an "In Review" column. These adjustments ensure that contributors are informed about the updated guidelines effectively, enhancing the overall development process and collaboration within the project.