generated from CDCgov/template
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Update minor dependencies to v5.4 #1339
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Quality Gate passedIssues Measures |
jorg3lopez
pushed a commit
that referenced
this pull request
Sep 27, 2024
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
jorg3lopez
added a commit
that referenced
this pull request
Sep 28, 2024
* Create 020_CA_ORU_R01_CDPH_OBX_to_LOINC_0_initial_message.hl7 Copied from the previous set of HL7 test data, with added OBX examples of the new mapping * Update 020_CA_ORU_R01_CDPH_OBX_to_LOINC_0_initial_message.hl7 Fixed typo on some local codes * Adding translated example file #20 * Addin skeleton for transformation * WIP - add initial implementation of MapLocalObservationCode transform * Adding unit test for a single observation * Adding test for local code mappable to a PLT * Update MapLocalObservationCodes.java Add the mapped coding at the beginning of the coding list * Small code cleanup * Add tests for when the initial code is a LOINC code * Refactor tests to remove duplicated code * Add assertion to evaluateCoding * Adding test for when primary code is also populated * Add tests for an unmapped local code and empty observation identifer * Updates to transformation to pass first 2 test cases * WIP - multiple observation test * All curent tests pass, but the code is not pretty and has at least one edge case unnacounted for * Complete the multiple observations test, add the remaining mappings to the lookup * Minor cleanup and refactoring * Replace for() loop with if check for a single coding on the observation * Add MapLocalObservationCodes to transformation_definitions.json * refactor: logging for when local codes not found added method to extract MSH10 to HapiHelper class * Update sender in log warning, restore mock logger in unit tests * Commenting out the PLT codes that haven't been mapped yet so they will cause warnings for now * refactor helper method for msh10 * Minor naming tweaks and comment updates * add null check for cwe extension * Address null check for getSystem method * Missing paren * refactor: added hascodingExtensionWithUrl helper method to HapiHelper added getCodingExtensionByUrl helper method to HapiHelper * refactor: added hasCodingSytem helper method to HapiHelper added getCodingSystem helper method to HapiHelper * Removed comments and empty lines * spotlessApply * WIP: transform refactoring - flatten conditional blocks * Refactor transform into simpler methods * Update minor dependencies to v5.4 (#1339) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update HapiHelper.java Replace unused assignment with inline return * Update dependency com.azure:azure-storage-blob to v12.28.0 (#1344) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Terraform azurerm to v4.3.0 (#1345) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Move urlForCodeType to HapiHelper for future reusability * Adding tests for code coverage and addressing Sonar findings * Adding tests for code coverage and addressing Sonar findings * Adding tests for null and empty bundle for resourcesInBundle * Adding tests for urlForCodeType * HapiHelper: unit test for when bundle is null * Updated RS Hurl Script to Replace MSH Header When Not Toggled (#1342) * Updating hurl scripts * Fixed typo in readme, create temp folder, write scrambled file into temp folder * Fixed directory creation, resolved issue with sed command not copying rest of file * updated readme to include toggle in options, add comments to scramble doing, alerted user to scrambling, removed folder creation for individual file * replaced msh header for oru and orm message types * added msh header replacement for oml message type * switched to msh-9.2 as subfield in conditional * remove temp directory after hurl script runs --------- Co-authored-by: jcrichlake <[email protected]> Co-authored-by: Sylvie <[email protected]> Co-authored-by: jcrichlake <[email protected]> Co-authored-by: saquino0827 <[email protected]> * Add control id test, update test name * Fix order of mockLogger registration and injection * Fixed prod prefix mapping Co-Authored-By: halprin <[email protected]> * Add post-TI and post-RS example files * Update dependency gradle to v8.10.2 (#1347) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Add final 5 PLT mappings; update mapping for 99717-50 * Add comment to clarify behavior of adding the mapped code --------- Co-authored-by: Joel Biskie <[email protected]> Co-authored-by: jorge Lopez <[email protected]> Co-authored-by: Tiffini Johnson <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Bella L. Quintero <[email protected]> Co-authored-by: jcrichlake <[email protected]> Co-authored-by: Sylvie <[email protected]> Co-authored-by: jcrichlake <[email protected]> Co-authored-by: saquino0827 <[email protected]> Co-authored-by: James Herr <[email protected]> Co-authored-by: halprin <[email protected]> Co-authored-by: Jorge Lopez <[email protected]>
tjohnson7021
pushed a commit
that referenced
this pull request
Oct 17, 2024
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
tjohnson7021
added a commit
that referenced
this pull request
Oct 17, 2024
* Create 020_CA_ORU_R01_CDPH_OBX_to_LOINC_0_initial_message.hl7 Copied from the previous set of HL7 test data, with added OBX examples of the new mapping * Update 020_CA_ORU_R01_CDPH_OBX_to_LOINC_0_initial_message.hl7 Fixed typo on some local codes * Adding translated example file #20 * Addin skeleton for transformation * WIP - add initial implementation of MapLocalObservationCode transform * Adding unit test for a single observation * Adding test for local code mappable to a PLT * Update MapLocalObservationCodes.java Add the mapped coding at the beginning of the coding list * Small code cleanup * Add tests for when the initial code is a LOINC code * Refactor tests to remove duplicated code * Add assertion to evaluateCoding * Adding test for when primary code is also populated * Add tests for an unmapped local code and empty observation identifer * Updates to transformation to pass first 2 test cases * WIP - multiple observation test * All curent tests pass, but the code is not pretty and has at least one edge case unnacounted for * Complete the multiple observations test, add the remaining mappings to the lookup * Minor cleanup and refactoring * Replace for() loop with if check for a single coding on the observation * Add MapLocalObservationCodes to transformation_definitions.json * refactor: logging for when local codes not found added method to extract MSH10 to HapiHelper class * Update sender in log warning, restore mock logger in unit tests * Commenting out the PLT codes that haven't been mapped yet so they will cause warnings for now * refactor helper method for msh10 * Minor naming tweaks and comment updates * add null check for cwe extension * Address null check for getSystem method * Missing paren * refactor: added hascodingExtensionWithUrl helper method to HapiHelper added getCodingExtensionByUrl helper method to HapiHelper * refactor: added hasCodingSytem helper method to HapiHelper added getCodingSystem helper method to HapiHelper * Removed comments and empty lines * spotlessApply * WIP: transform refactoring - flatten conditional blocks * Refactor transform into simpler methods * Update minor dependencies to v5.4 (#1339) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update HapiHelper.java Replace unused assignment with inline return * Update dependency com.azure:azure-storage-blob to v12.28.0 (#1344) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Update Terraform azurerm to v4.3.0 (#1345) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Move urlForCodeType to HapiHelper for future reusability * Adding tests for code coverage and addressing Sonar findings * Adding tests for code coverage and addressing Sonar findings * Adding tests for null and empty bundle for resourcesInBundle * Adding tests for urlForCodeType * HapiHelper: unit test for when bundle is null * Updated RS Hurl Script to Replace MSH Header When Not Toggled (#1342) * Updating hurl scripts * Fixed typo in readme, create temp folder, write scrambled file into temp folder * Fixed directory creation, resolved issue with sed command not copying rest of file * updated readme to include toggle in options, add comments to scramble doing, alerted user to scrambling, removed folder creation for individual file * replaced msh header for oru and orm message types * added msh header replacement for oml message type * switched to msh-9.2 as subfield in conditional * remove temp directory after hurl script runs --------- Co-authored-by: jcrichlake <[email protected]> Co-authored-by: Sylvie <[email protected]> Co-authored-by: jcrichlake <[email protected]> Co-authored-by: saquino0827 <[email protected]> * Add control id test, update test name * Fix order of mockLogger registration and injection * Fixed prod prefix mapping Co-Authored-By: halprin <[email protected]> * Add post-TI and post-RS example files * Update dependency gradle to v8.10.2 (#1347) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> * Add final 5 PLT mappings; update mapping for 99717-50 * Add comment to clarify behavior of adding the mapped code --------- Co-authored-by: Joel Biskie <[email protected]> Co-authored-by: jorge Lopez <[email protected]> Co-authored-by: Tiffini Johnson <[email protected]> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Bella L. Quintero <[email protected]> Co-authored-by: jcrichlake <[email protected]> Co-authored-by: Sylvie <[email protected]> Co-authored-by: jcrichlake <[email protected]> Co-authored-by: saquino0827 <[email protected]> Co-authored-by: James Herr <[email protected]> Co-authored-by: halprin <[email protected]> Co-authored-by: Jorge Lopez <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
5.3.1
->5.4
5.3.1
->5.4
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.