Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: implementacion de la operacion delete #14

Merged
merged 3 commits into from
May 12, 2021
Merged

Conversation

EmiRetamar
Copy link
Contributor

@EmiRetamar EmiRetamar commented May 10, 2021

Descripcion

Se implemento la funcion delete para la cache, en donde se verifica si existe la key recibida, en caso existir se elimina de la estructura de datos y en caso contrario se devuelve un error

Issue Relacionada

https://github.com/gophers-latam/GoKey/projects/1#card-60712589

Motivacion y Contexto

Es parte de las operaciones de la cache, resuelve el problema de poder quitar un elemento de la misma

Como fue probado

Aun no se desarrollaron los tests para esta funcionalidad

Screenshots / capturas de pantalla (si es necesario)

Tipo de cambio

  • Bug fix (non-breaking cambios que fixean una issue)
  • Nueva feature / funcionalidad (non-breaking cambio que agrega una nueva funcionalidad)
  • Breaking change (fix o feature que va a causar un cambio en una funcionalidad existente)

Checklist

  • Estas haciendo el pull request desde un *topic/feature/bugfix branch (lado derecho). Si estas haciendo un pull request desde un fork, no lo hagas desde master!.
  • Estas haciendo el pull request contra master (lado izquierdo). Tambien de que estas usando los ultimos cambios en master.
  • Mis cambios necesitan cambio de la documentacion.
  • Actualize la documentacion acordemente.
  • Modules and dependencias fueron actualizadas acordemente; correr go mod tidy && go mod vendor
  • Agregue tests para cubrir mis cambios.
  • Todos los tests existentes pasaron.
  • Checkear que el codigo que estoy subiendo esta linteado:
    • go fmt -s
    • go vet

Copy link
Member

@tomiok tomiok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agregar tests de delete

@EmiRetamar
Copy link
Contributor Author

Dale a la noche agrego los tests

@EmiRetamar EmiRetamar changed the title Implementacion de la operacion delete feat: implementacion de la operacion delete May 11, 2021
@tomiok tomiok merged commit 6310440 into main May 12, 2021
@tomiok tomiok deleted the delete-implementation branch May 12, 2021 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants