Contains long running updates to docs, minor process improvements, and a refactoring of code to a more consistent and readable naming convention.
Additions
see full PR for changes.
Removals
see full PR for changes.
Changes
Naming convention for several functions was altered to be more cleanly readable
Testing
To test for leaked keys, I ran grep -Rn "AWS_ACCESS_KEY" /path/to/RRASSLER and returned 0 lines that were not intended to be included here
Screenshots
NA
Notes
This is hard to test, since the formal rendering of the pages is tied to the dev branch and dev branch artifacts, and so many urls and images won't render until merged. I am happy to show you how this is run locally if that would help.
Todos
See the discussion section. Misc. and partial functions may not be complete and are flagged as 'dev'
Checklist
[X] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows project standards (link if applicable)
[ ] Passes all existing automated tests
[X] Any change in functionality is tested
[X] New functions are documented (with a description, list of inputs, and expected output)
[X] Placeholder code is flagged / future todos are captured in comments
[X] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[X] Reviewers requested with the Reviewers tool :arrow_right:
Testing checklist
Target Environment support
[X] Windows (via installed package in windows side)
Contains long running updates to docs, minor process improvements, and a refactoring of code to a more consistent and readable naming convention.
Additions
Removals
Changes
Testing
grep -Rn "AWS_ACCESS_KEY" /path/to/RRASSLER
and returned 0 lines that were not intended to be included hereScreenshots
NA
Notes
Todos
Checklist
Testing checklist
Target Environment support
Accessibility
Other