BREAKING: Refactor script execution summary
@jmgate is already working on this.
Since Sep 5, 2024.
Assessment
This issue has not been assessed yet.
Description
I'd like the script execution summary to be more customizable. I'd like the base class to provide a variety of SummaryItems (for lack of a better name). Subclass developers can extend/override what's provided by the base class, and they can create their own. When they call print_script_execution_summary(), they can pass it a list of summary items for it to process, or it can fall back to the default list provided by the base class.
Note: This will likely be a breaking change, though it might be possible to provide a backward compatibility layer for a time.
- Dominant language
- Python
- Stars
- 23
- Forks
- 2
- Avg merge
- 5d 4h
- Merged PRs (30d)
- 3
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from sandialabs/staged-script
-
Quick Type: Refactor
Difficulty 3/5 1-2 days Newbie friendliness 35/100
sandialabs/staged-script#300 · 1 comment ·
-
Type: Feature
Difficulty 4/5 3-5 days Newbie friendliness 35/100
sandialabs/staged-script#172 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 25/100
sandialabs/staged-script#139 ·
-
Add Trilinos build scriptMay be free again @jmgate claimed this 767 days ago, and no pull request is open. OpenType: Documentation
sandialabs/staged-script#98 · 1 comment · 1 assignee ·
-
Save script execution summaryMay be free again @jmgate claimed this 767 days ago, and no pull request is open. OpenType: Feature
sandialabs/staged-script#97 · 1 assignee ·
All issues in sandialabs/staged-script
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Graphify-Labs/graphify#4524 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
topoteretes/cognee#5982 ·
Maintainers usually reply within 1 day
-
Patches of derivations with `__structuredAttrs` are ignoredPossibly taken @KiaraGrouwstra claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 81/100
nix-community/vulnix#184 ·
-
New InternshipOpennew_internship
Difficulty 1/5 Under an hour Newbie friendliness 65/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 64/100
aws-samples/sample-per-model-bedrock#295 ·
Maintainers usually reply within 1 day