-
Notifications
You must be signed in to change notification settings - Fork 195
Create early-and-fluid-quality-management-for-predictable-delivery #498
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
Open
Trueblueaddie
wants to merge
8
commits into
InnerSourceCommons:main
Choose a base branch
from
Trueblueaddie:patch-3
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 5 commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
95fdd1e
Create early-and-fluid-quality-management-for-predictable-delivery
Trueblueaddie 59b0b27
Apply suggestions
Trueblueaddie 2a8df4e
Add markdown extension to filename
spier f7dac0f
Add spacing before header
spier 7ddd62d
Remove trailing spaces
spier eaebb18
Removing trailing space
spier d3e4a9d
Add initial Context
spier 9fcbdea
Spacing
spier File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
60 changes: 60 additions & 0 deletions
60
...rns/2-structured/early-and-fluid-quality-management-for-predictable-delivery.md
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,60 @@ | ||
## Title | ||
|
||
Early and fluid quality management for predictable delivery | ||
|
||
## Patlet | ||
|
||
Testing practices are often short-cutted to meet deadlines leading to stability questions regarding the code. Frequent code review and testing is embedded in a smaller pull-request culture. | ||
|
||
## Problem | ||
|
||
Code stability depends on how confident we are on our tests. In some cases, proper testing is not taking place until later stages (even pre-production deployment), generating delays in the delivery of the releases. Part of the problem is the size of changes. Bigger changes are more difficult to review and test. In other words, our development code is sent to pre-production without passing the tests, creating blocks and loops against pre-production environments while fixing the issues. | ||
|
||
The lack of automated testing coverage leaves a lot of room for fears of breaking things (regressions), which in turn tend to prevent contributions and their acceptance. The contributions are also more error-prone and thus need more corrective effort. This delays them and increases the risk of conflicting with other code changes. | ||
|
||
## Forces | ||
spier marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
Against: | ||
Time pressures by the business team. | ||
Needs investments to reap benefit cycle after cycle. | ||
|
||
In favor: | ||
Agility is a trend in both software development and project management (lean) | ||
Perfectionism on the technical teams. | ||
Lack of confidence in the master branch of development, lack of stability. | ||
|
||
## Solutions | ||
|
||
Automated testing, routine code review, and frequent pull request processes are put in place to ensure quality. | ||
|
||
Planning: Break the delivery in smaller chunks (minimum viable changes). This goes also for processes and tooling. | ||
Coordination: Agree on outcome validation criteria upfront. | ||
Culture: Define needs and outcome, not the solution | ||
Culture: Approve as soon as situation improves (don’t wait for perfection) and iterate to improve | ||
Culture: Stakeholders watch and participate in development according to their capabilities (pull request culture). | ||
Automate testing to reduce cycle times and repetitive effort | ||
|
||
## Resulting Context | ||
|
||
Since changes are small and review is very automated, reviews are swift and problems are detected early, so they are fixed quickly. | ||
Early and flawless delivery of key functionality provides and hardens confidence in the development. | ||
Common pull request culture enables other stakeholders to provide and validate solution proposals. Development accelerates with InnerSource contributions. | ||
|
||
## Known Instances | ||
|
||
* Santander Bank | ||
|
||
## Status | ||
spier marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
Initial | ||
Published internally in Santander Bank; drafted via pull-request in December of 2022 | ||
|
||
## Authors | ||
|
||
Alberto Pérez García-Plaza | ||
Daniel Izquierdo Cortázar | ||
Addie Girouard | ||
|
||
## Acknowledgements | ||
|
||
Igor Zubiaurre |
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.
Uh oh!
There was an error while loading. Please reload this page.