Skip to content
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

Refactor script to add safeguarding to flows for v2 #8

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

istride
Copy link
Contributor

@istride istride commented Nov 1, 2023

The script uses idioms and conventions from Python which would be unusual and surprising to a JavaScript developer. Commented and unused code has been removed and certain sections have been rewritten for the sake of legibility. The behaviour of the script should be unchanged.

The script uses idioms and conventions from Python which would be unusual and
surprising to a JavaScript developer. Commented and unused code has been removed
and certain sections have been rewritten for the sake of legibility. The
behaviour of the script should be unchanged.
@istride istride requested a review from edmoss345 November 1, 2023 15:58
Copy link
Collaborator

@edmoss345 edmoss345 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have tested this by running the pipeline before and after the refactor and after reviewing the output from the safeguarding step it looks like we get the same result. Therefore think this is good to merge as it appears to have the same functionality as the existing process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants