Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Fix: #3876 Update task utils: removeIterationFromTaskRefName #3878

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Fix: #3876 Update task utils: removeIterationFromTaskRefName #3878

wants to merge 5 commits into from

Commits on Dec 3, 2023

  1. Fix: Netflix#3876 Update task utils: removeIterationFromTaskRefName 

    Parsing name is not considering task ref name with double underscores

- This is not fully fixing the use of this function. There needs to be some kind of validation against the user from setting up the taskRefName with double underscore
    @appunni-m
    appunni-m committed Dec 3, 2023
    Configuration menu
    Copy the full SHA
    eeca96a View commit details
    Browse the repository at this point in the history

  2. Merge pull request #1 from appunni-m/fix-3876-do-while-infinite-loop 

    Fix: Netflix#3876 Update task utils: removeIterationFromTaskRefName
    @appunni-m
    appunni-m committed Dec 3, 2023
    Configuration menu
    Copy the full SHA
    c685368 View commit details
    Browse the repository at this point in the history

Commits on Dec 4, 2023

  1. Update TaskUtils.java 

    Added imports as required
    @appunni-m
    appunni-m committed Dec 4, 2023
    Configuration menu
    Copy the full SHA
    884a9fe View commit details
    Browse the repository at this point in the history

  2. Check for __$i in the reference name

    @appunni-m
    appunni-m committed Dec 4, 2023
    Configuration menu
    Copy the full SHA
    dec43ea View commit details
    Browse the repository at this point in the history

  3. Merge pull request #2 from appunni-m/main 

    Merge Netflix#3880 fix
    @appunni-m
    appunni-m committed Dec 4, 2023
    Configuration menu
    Copy the full SHA
    dd00723 View commit details
    Browse the repository at this point in the history