-
Notifications
You must be signed in to change notification settings - Fork 195
Create fixing-issues-step-by-step.md #503
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
9
commits into
InnerSourceCommons:main
Choose a base branch
from
Trueblueaddie:patch-8
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 8 commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
faf0bd9
Create fixing-issues-step-by-step.md
Trueblueaddie 50f4893
Update fixing-issues-step-by-step.md
Trueblueaddie 9a9e833
Rewording
spier 4771d5e
Adding initial Context section
spier 13d06c3
Fix spacing
spier 155dfa9
Apply suggestions
Trueblueaddie 16c422a
Add spacing before header
spier 23ae945
Add spacing before header
spier bb0fd61
Removing trailing space
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
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,58 @@ | ||
## Title | ||
|
||
Fixing Issues Step by Step | ||
|
||
## Patlet | ||
|
||
Solving an issue by deadline without consideration of the current scope of change to process causes developer frustration and decreased efficiencies. Utilize a continuous improvement approach which right-sizes the work and provides realistic expectations. | ||
A baby-step development culture fosters external contributions (more opportunities for engaging, less to learn, less to comply with, less to document, less to review). This culture reflects in the open source mantra _["Release early, release often"](https://en.wikipedia.org/wiki/Release_early,_release_often)_. | ||
|
||
## Problem | ||
spier marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
Results are expected to take place within a few days/weeks after a process has started. Adding a process or a toolchain will not provide any result by itself beyond developer frustration. Stating that something has to work by a certain deadline will not make it possible. This will just force the situation, and workarounds will be done by developers. | ||
|
||
## Context | ||
|
||
* Developer frustration is spread and at high levels. Complaints about pressure and stress. | ||
* Clumpsy workarounds, undocumented and/or prone to be broken/voided are frequent. | ||
* Voluntary contributions are scant. | ||
spier marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
spier marked this conversation as resolved.
Show resolved
Hide resolved
|
||
## Forces | ||
|
||
Understanding step changes in a process requires adequate time and clarity of impact. | ||
Lack of clarity of impact on the process and delivery pressure are deterrents to change. | ||
|
||
## Solutions | ||
|
||
When planning new steps, set the way that step is successful if compared to others. Follow a continuous improvement approach and measure the delta between steps. | ||
Splitting targets into smaller bits multiplies the opportunities for engagement. And they are more attractive too, because they require less to learn, less to comply with, less to document, and less to review (reviews are contributions too). | ||
|
||
When reviewing contributions, don't strive for perfection and approve them as soon as there's a tangible improvement (regressions must be avoided, of course). | ||
|
||
Lurkers will feel that the (small) achievement is within their reach and will convince them to contribute. This bootstraps a virtuous cycle, as more frequent and diverse contributions also seduce other lurkers to join the party. | ||
spier marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
Trueblueaddie marked this conversation as resolved.
Show resolved
Hide resolved
|
||
## Resulting Context | ||
spier marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
Expectations are aligned and realistic in terms of the improvement path of the organization. A continuous improvement is required and understood. | ||
Trueblueaddie marked this conversation as resolved.
Show resolved
Hide resolved
|
||
Contributors feel like they can contribute partial improvements without the need to solve all problems at once. | ||
Contributions are small and can be reviewed swiftly. | ||
Trusted Committers accept contributions or partial or imperfect solutions if they provide enough value to improve the previous status. Pointers on how to perfect the contribution become comments instead of mandates. | ||
|
||
## Known Instances | ||
|
||
Santander Bank | ||
|
||
## Status | ||
|
||
Structured | ||
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.