Remove prevent_destroy on S3 bucket

I understand this was introduced for parity with CFN but unlike CFN "Retain" policy, setting this on S3 bucket resource effectively prevents the module user from being able to destroy _anything_ in the stack (without forking the module and removing prevent_destroy).

Author: oavdeev

aws/terraform/modules/metaflow/modules/datastore/s3.tf

@@ -16,10 +16,6 @@ resource "aws_s3_bucket" "this" {
       Metaflow = "true"
     }
   )
-
-  lifecycle {
-    prevent_destroy = true
-  }
 }
 
 resource "aws_s3_bucket_public_access_block" "this" {