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

@gajaguar<Gerardo Antonio Gerónimo Vasconcelos> #62

Open
wants to merge 101 commits into
base: master
Choose a base branch
from

Conversation

gajaguar
Copy link

This is my solution for the pre cohort Platzi Master challenge.
I turn the project into typescript, it mean breaking changes on each comit, lol.
I try implement a little bit of SOLID principles. The test just breaks, no time for fix it.
My efforts were left in this project. I hope it be of your liking.

gajaguar added 30 commits July 23, 2021 19:41
- Create `build`.
- Change `dev` to work with typescript.
- Add `lint:js` to run `eslint`.
- Add `lint` as shortcut of `lint:js`.
- Change `start` to execute from `lib` directory.
- Add `test` as shortcut of `test:e2e`.
gajaguar added 29 commits July 27, 2021 23:03
Move the builder to `utils` directory and convert into a class.
Fix an error that cause that only the first registered resource
endpoints works.
- Category
- Product
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.

1 participant