-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Maintainers Check List
Carrie Roberts edited this page Jun 10, 2021
·
17 revisions
Maintainers Check List
- Is the atomic in the right Technique #
- Is it a duplicate of an atomic that already exists
- Make sure Github links within commands that are executed use the permanent link if not in the redcanary repo
- Input arguments used in the prereqs, commands, and cleanup commands (not hardcoded if defined in input args).
- Supported files included in /bin (for binary) or /src (for human readable)
- Are supported files included in the PR? Source code should be provided where custom binaries are provided (if possible)
- Cleanup commands are intended to remove temporary files, and reset system enough to run the test again but shouldn't remove prerequisites. Code should allow the cleanup commands to be run multiple times in a row without generating errors/warnings.
- Test names are descriptive. Descriptions explain what the atomic does and what success looks like. It should include links to relevant blogs and tools where applicable.
- Input arg defaults allow atomic to function on most systems without modification.
- Atomic test changes/additions are made to yaml file (and not md file since md file gets rewritten from the yaml by the Circle CI process at merge time)
Lead Reviewer Assignments:
- Linux/Macos - Carl (rc-grey on GitHub, @Carl Petty on Slack)
- Cloud - Bhavin (patel-bhavin on Github, @Bhavin Patel on Slack)
- Windows - Carrie Roberts (clr2of8 on GitHub, @OrOneEqualsOne on Slack)
Questions? Get connected to the community on the Atomic Red Team™ Slack channel.