-
Notifications
You must be signed in to change notification settings - Fork 14
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
SPINEDEM-3725 AC2 #1046
base: master
Are you sure you want to change the base?
SPINEDEM-3725 AC2 #1046
Conversation
This branch is working on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket: SPINEDEM-3725 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will need to figure out why the pipelines are not running.
|
||
Special care should be taken when performing multiple removals in the same list; as removing a particular index could affect all subsequent index positions. The next two examples perform **<u>exactly</u>** the same operation. | ||
Special care should also be taken when performing multiple removals in the same list; as removing a particular index could affect all subsequent index positions. The next two examples perform **<u>exactly</u>** the same operation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a recommendation here to perform updates to a list in reverse order to avoid this complication?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A bit further down this section it has such a recommendation:
Which is the intended outcome. Providing the patches with the indexes descending means that the list stays in a stable format the whole way through as the only changes to the index positions are items have been passed over already. |
This branch is working on a ticket in the NHS Digital APM JIRA Project. Here's a handy link to the ticket: SPINEDEM-3725 |
Summary
Reviews Required
Review Checklist
ℹ️ This section is to be filled in by the reviewer.