[
    {
        "line_no": 15,
        "created_by": "azure-sdk",
        "comment_text": "Enum member names should be consistently uppercase.\\n\\nSuggestion : V7_7_PREVIEW_1 = \"7.7-preview.1\"",
        "created_on": "2023-10-01T12:00:00Z"
    },
    {
        "line_no": 61,
        "created_by": "azure-sdk",
        "comment_text": "Rename the class to remove the redundant 'Object' suffix for clarity.\\n\\nSuggestion : class azure.keyvault.secrets.RollSecretParameters:",
        "created_on": "2023-10-01T12:00:00Z"
    },
    {
        "line_no": 250,
        "created_by": "azure-sdk",
        "comment_text": "The async client’s deletion method does not mirror the sync client’s naming (which uses 'begin_delete_secret'); both should use the same method name across sync and async clients.\\n\\nSuggestion : async def begin_delete_secret(\\n\\nSee: https://azure.github.io/azure-sdk/python_design.html#python-client-separate-sync-async",
        "created_on": "2023-10-01T12:00:00Z"
    }
]
