Skip to content

Bubeniuk Vlad #26

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

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

Conversation

ZayJob
Copy link

@ZayJob ZayJob commented Nov 9, 2019

The process of executing the final task.

  • [Iteration 1] One-shot command-line RSS reader.
  • [Iteration 2] Distribution.
  • [Iteration 3] News caching.
  • [Iteration 4] Format converter.
    • Converting to HTML format
    • Converting to PDF format
  • * [Iteration 5] Output colorization.
  • * [Iteration 6] Web-server. (check iteration_6 branch,very low functionality)

Additionally

  • Pycodestyle
  • Unit tests(смотрите документацию))
  • Documentation
  • Docker and Docker-compose

Create virtualenv python3.8 and add README.md file
--limit
--verbose
--version
--help
--json
Rss and application modules redone,
added json scheme and annotation methods
And also checked the code through the paycodestyle
@AlexeiBuzuma AlexeiBuzuma added the [Deadline] Iteration 1-2 This is a marker for first and second iterations deadline. label Nov 17, 2019
ZayJob and others added 8 commits November 18, 2019 00:57
Added docker-compose:
	-docker-compose.yml
	-Dockerfile
Тew parameter added:
	--date - The cashed news can be read with it. The new from the specified day will be printed out.
		 If the news are not found return an error.
Add services mongo and mongo-express in docker-compose.yml
Add mongodb.py file and doc string for work with database
Update requirement.txt file
Update version proj.
Type annotation changed
Change __init__ in MongoDatabase class

from rss_reader_ft import config

setup(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Не поставились зависимости после установки пакета

# rss-reader --help
Traceback (most recent call last):
  File "/usr/local/bin/rss-reader", line 5, in <module>
    from rss_reader_ft.rss_reader import main
  File "/usr/local/lib/python3.8/site-packages/rss_reader_ft/rss_reader.py", line 4, in <module>
    from rss_reader_ft.app.application import Application
  File "/usr/local/lib/python3.8/site-packages/rss_reader_ft/app/application.py", line 7, in <module>
    from rss_reader_ft.rss.data_loader import DataLoader
  File "/usr/local/lib/python3.8/site-packages/rss_reader_ft/rss/data_loader.py", line 4, in <module>
    import feedparser
ModuleNotFoundError: No module named 'feedparser'

Copy link
Collaborator

Choose a reason for hiding this comment

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

# rss-reader --help
Traceback (most recent call last):
  File "/usr/local/bin/rss-reader", line 5, in <module>
    from rss_reader_ft.__main__ import main
  File "/usr/local/lib/python3.8/site-packages/rss_reader_ft/__main__.py", line 2, in <module>
    from rss_reader_ft import rss_reader
  File "/usr/local/lib/python3.8/site-packages/rss_reader_ft/rss_reader.py", line 4, in <module>
    from rss_reader_ft.app.application import Application
  File "/usr/local/lib/python3.8/site-packages/rss_reader_ft/app/application.py", line 9, in <module>
    from rss_reader_ft.rss.output import Output
  File "/usr/local/lib/python3.8/site-packages/rss_reader_ft/rss/output.py", line 9, in <module>
    from rss_reader_ft.conversion.pdf_converter import PdfConverter
  File "/usr/local/lib/python3.8/site-packages/rss_reader_ft/conversion/pdf_converter.py", line 7, in <module>
    from reportlab.pdfbase.ttfonts import TTFont
ModuleNotFoundError: No module named 'reportlab'

from rss_reader_ft import config

setup(
name=config.__package__,
Copy link
Collaborator

Choose a reason for hiding this comment

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

после установки зависимостей утилита замирает при попытке ее запустить, verbose не дает информации полезной о происходящем

# rss-reader https://news.tut.by/rss/geonews/minsk.rss --verbose

Copy link
Author

@ZayJob ZayJob Nov 27, 2019

Choose a reason for hiding this comment

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

Возможно, ты пропустил пункт, где надо было откатиться до старого коммита, и запускал последнюю версию, где надо поднимать бд(не могу настроить правильно docker-compose.yaml, был бы благодарен в помощи по его настройке), я обновил ридми файл по установке 2 итерации и протестировал в чистом докере

Copy link
Author

Choose a reason for hiding this comment

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

добавил документацию по использованию 3-5 итерации

@ZayJob ZayJob changed the title [WIP] Bubeniuk Vlad Bubeniuk Vlad Dec 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants