Code Monkey home page Code Monkey logo

graduate_work's Introduction

Hi there 👋 I'm Stanislav

Skills And Technologies | Currently Learning | Plans To Learn
Skills And Technologies 💻 I’m Currently Learning 🌱 Plans To Learn 📖
- git.png Git - elastic.png Elasticsearch - Oauth2
- python.png Python - python.png Asyncio - kafka.apache.png Apachekafka
- django.png Django(django-rest-framework.png DRF)/flask.png Flask/fastapi.png FastAPI - Elk
- nginx.png Nginx
- favicon.ico Mssql/postgresql.png PostgreSQL/sqlite icon Sqlite/redis.png Redis
- docker.png Docker/DockerCompose
- makefile icon Make
- Etl
- swagger.png Swagger
- rabbitmq.png Rabbitmq
Coding Activity (7 days) Languages (all time)
Other stats
Codewars GitHub Streak GitHub Stats
codewars GitHub Streak Matwave's github stats
How to reach me 📫

graduate_work's People

Contributors

answerkas avatar matwave avatar py-smith avatar

Watchers

 avatar

graduate_work's Issues

Code review

Привет! Хорошая работа! Все замечания ниже - мелкие, но они все-таки есть:

  1. Сделайте ваш репозиторий приватным (согласно оферте вы не можете выкладывать в паблик свои работы) и не забудьте отправить приглос нашу юзеру
  2. Эта зависимость deprecated. aioredis переехал в redis-py, подробнее: https://github.com/aio-libs/aioredis-py
  3. nit: в этом месте лучше указывать всех авторов :)
  4. Можно лучше: эти хуки уже deprecated. В этой же статье новый вариант использования.
  5. nit: было бы лучше этот эндпоинт назвать alice
  6. А почему здесь не наследуетесь от BaseOrjsonModel?
  7. С версии python 3.9 для стандартных коллекций больше не нужен модуль typing: https://docs.python.org/3.9/whatsnew/3.9.html#type-hinting-generics-in-standard-collections. Поправьте во всем проекте
  8. Такие комментарии логичнее поместить в документацию метода
  9. А как этот лог из контейнера будет попадать куда-либо?) Лучше писать просто в stdout/stderr (то есть в консоль) и docker сам весь вывод забирает и позволяет ротировать. Пример для docker-compose: https://docs.docker.com/config/containers/logging/configure/#configure-the-default-logging-driver
  10. Тут можно проще:
return any(intent.startswith(text_request) for intent in command)
  1. Код внутри if лучше вынести в отдельные методы. При увеличении функционала это дерево if будет нечитаемым.
  2. Этот метод и все остальные, связанные с взаимодействием с апишкой фильмов должны быть в отдельном базом сервисе. Иначе вам придется для каждого помощника поддерживать свои интерфейсы, что сильно затруднит поддержку
  3. Тут не используете fastapi.status
  4. Тут больше двух пустых строк между методами - запустите любой линтер и поправьте код в соответствии с PEP8
  5. nit: Опечатка в названии файла
  6. print в коде быть не должно
  7. Нет смысла в этом, на следующей строчке всегда вернете ответ
  8. У вас же не может быть двух команд одновременно? Тогда эти if стоит превратить в elif
  9. Этот метод очень большой, давайте декомпозировать

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.