From 8e276600b5a68a3152f00cbdd6b9b9c31c637ed3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carlos=20Michel=20Mourra=20D=C3=ADaz?=
 <60996279+MichMourra@users.noreply.github.com>
Date: Thu, 8 Sep 2022 22:30:18 -0500
Subject: [PATCH] Sugerencias Tarea 1
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Excelente código :), cumple correctamente con todos los requisitos solicitados en la tarea. De igual manera realizaste correctamente el uso de las tags, el manejo de Issues y utilizaste dos ramas en GitHub. Tu manejo de las expresiones regulares es increíble, sigue así 🥇. Solamente me gustaría darte dos recomendaciones, la primera es que te asegures de eliminar los saltos de linea al momento en el que extraes tu secuencia, esto lo puedes hacer con un rstrip. Por otro lado, considero que tus docstrings están muy bien hechos, sin embargo creo que seria de mucha ayuda que agregaras un poco mas de lineas de documentación para entender mejor la lógica que empleaste en algunos pasos.
---
 src/rich_at.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/rich_at.py b/src/rich_at.py
index 5faf74d..738e9ac 100644
--- a/src/rich_at.py
+++ b/src/rich_at.py
@@ -47,7 +47,7 @@
 
 # Abrimos el archivo y extraemos su contenido
 with open(args.file, "r") as seq_file:
-    dna = seq_file.read().upper()
+    dna = seq_file.read().rstrip('\n').upper()
 
 def evaluate(dna):
     '''
@@ -95,4 +95,4 @@ def find_regions(dna, at=2):
     if args.search:
         ans = find_regions(dna,args.search)
     else:
-        ans = find_regions(dna)
\ No newline at end of file
+        ans = find_regions(dna)