is this coorect? I think i took it the wrong way (but shows the same result)

# TODO Create an empty list to maintain the player names
players = []


# TODO Ask the user if they'd like to add players to the list.
# If the user answers "Yes", let them type in a name and add it to the list.
# If the user answers "No", print out the team 'roster'
while True:
    entry = input("Would you like to add player (yes/no)  ")
    if entry == "yes":
        player_name = input("Name  ")
        players.append(player_name)
    elif  entry == "no":
        if len(players) >= 1:
            number = 1
            print("Your Team: ")
            for player in players:
                print("player {}: {}".format(number, player))
                number += 1
            break
        else:
            print("roster")
            break

if len(players) >= 1:
        # TODO print the number of players on the team
    print("your team has {} players".format(len(players)))

    # TODO Print the player number and the player name
    # The player number should start at the number one


    # TODO Select a goalkeeper from the above roster
    goalkeeper = int(input("Who will be your goal keeper? (1-{})  ".format(len(players))))
    # TODO Print the goal keeper's name
    # Remember that lists use a zero based index
    print("Your goal keeper will be {}".format(players[goalkeeper-1]))