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.

Sign up for GitHub

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

Jump to bottom

Validations #77

Merged
merged 18 commits into from
Apr 24, 2022
Merged

Validations #77

merged 18 commits into from
Apr 24, 2022

Conversation

PalumboN
Copy link
Collaborator

@PalumboN PalumboN commented Apr 18, 2022 •
edited
Loading

Fix #41

  • Update a Wollok 3.0.8
  • Se corren las validaciones.
  • Se muestran en cada elemento del AST y en un listado general.
  • Se testea y agregan mocks

listado

module

Cargar issues

  • Traducir los código de error -> Plantear una forma de reutilizarlos en el IDE Traducir las validaciones #81
  • Ahora al editar el body de un método o test, las validaciones no se corren hasta que "confirmás" (☑️) todo el código (y salís de la edición). Estaría bueno tener las validaciones en el momento en que se edita el método. Quizá se podría plantear un "auto-save" y "confirmar" en cada edición? Editar un Body #82
  • Armar una Wiki contando a groso modo el diseño de la app (componentes más importantes, estructura del proyecto, features de Wollok, etc.)

@PalumboN PalumboN requested a review from ivojawer April 18, 2022 16:28
@PalumboN PalumboN mentioned this pull request Apr 18, 2022
Merged
2 tasks
ivojawer
ivojawer approved these changes Apr 21, 2022
View reviewed changes
Copy link
Collaborator

@ivojawer ivojawer left a comment

Choose a reason for hiding this comment

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

Qui locura

src/components/projects/ProjectHeader.tsx Outdated Show resolved Hide resolved
@PalumboN PalumboN merged commit e81e444 into dev Apr 24, 2022
@PalumboN PalumboN deleted the validations branch April 24, 2022 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ivojawer ivojawer ivojawer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Correr validaciones y mostrarlas
2 participants
@PalumboN @ivojawer