Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
vsmutok created a review comment on a pull request on mate-academy/py-linter-formatter
use `format_linter_error` here
vsmutok created a review comment on a pull request on mate-academy/py-linter-formatter
no need to use loop here
vsmutok created a review comment on a pull request on mate-academy/py-linter-formatter
use `format_single_linter_file` here
vsmutok created a comment on a pull request on mate-academy/py-car-wash-station
> Спершу ШІ виділив у мене таку помилку: > > self.count_of_ratings += 1 self.average_rating = round((self.average_rating * (self.count_of_ratings - 1) + single_rate) / self.count_of_ratings, 1) ...
vsmutok created a review on a pull request on mate-academy/py-fastapi-city-temperature-management-api
Great!
vsmutok created a review comment on a pull request on mate-academy/py-tickets-and-orders
use transaction.atomic decorator for better readability
vsmutok created a review comment on a pull request on mate-academy/py-tickets-and-orders
use `get_user` here
vsmutok created a review comment on a pull request on mate-academy/py-tickets-and-orders
The `get` method will raise a `DoesNotExist` exception if the user is not found. Consider handling this exception to avoid runtime errors.
vsmutok created a review on a pull request on mate-academy/py-car-wash-station
Please fix `flake8` error!
vsmutok created a review comment on a pull request on mate-academy/py-scrape-quotes
remove this comment
vsmutok created a review on a pull request on mate-academy/py-create-file-from-terminal
1. The logic doesn't work as expected: an extra folder `-d` is created, and the file name is incorrect. 2. If I run the command a second time, it causes an error. Tip: Read about the `argpa...
vsmutok created a review on a pull request on mate-academy/py-linter-formatter
Please fix `flake8` errors
vsmutok created a review comment on a pull request on mate-academy/py-create-file-from-terminal
```suggestion if args.f is None or args.d is not None: create_dirs() if args.d is not None: os.chdir(os.path.join(*args.d)) if args.f is not None: create_file() ```
vsmutok created a review comment on a pull request on mate-academy/py-create-file-from-terminal
1. **Readability**: Independent blocks are easier to understand than nested structures. 2. **Avoids nesting**: No deep indentation, making the code cleaner.
vsmutok created a review comment on a pull request on mate-academy/py-create-file-from-terminal
This will look somewhat better. :) ```suggestion if args.f is None or args.d is not None: create_dirs() if args.d is not None: os.chdir(os.path.join(*args.d)) if args.f is not Non...
vsmutok created a review comment on a pull request on mate-academy/py-create-file-from-terminal
There should be a blank line between entries in the file. 
vsmutok created a review comment on a pull request on mate-academy/py-create-file-from-terminal
Try to use `with` only once and open the file just once.
vsmutok created a review comment on a pull request on mate-academy/py-create-file-from-terminal
```suggestion line = input(f"Enter content line: Line{count} content: ") ```
vsmutok created a review comment on a pull request on mate-academy/py-create-file-from-terminal
Try to use `with` only once and open the file just once. It's also important that the data should be appended to the file, not overwritten.