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

Pull request file-manager #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Pull request file-manager #1

wants to merge 4 commits into from

Conversation

Vitaljano
Copy link
Owner

@Vitaljano Vitaljano commented Jun 12, 2022

1.Task: File-manager

2.Done 12.06.22 (deadline 20.12.22)
3.Score: 330 / 330

Basic Scope

  • General -+6 Application accepts username and prints proper message -+10 Application exits if user pressed ctrl+c or sent .exit command and proper message is printed

Operations fail

  • +20 Attempts to perform an operation on a non-existent file or work on a non-existent path result in the operation fail
  • +10 Operation fail doesn't crash application

Navigation & working directory operations implemented properly

  • +10 Go upper from current directory
  • +10 Go to dedicated folder from current directory
  • +10 List all files and folders in current directory

Basic operations with files implemented properly

  • +10 Read file and print it's content in console
  • +10 Create empty file
  • +10 Rename file
  • +10 Copy file
  • +10 Move file
  • +10 Delete file

Operating system info (prints following information in console) implemented properly

  • +6 Get EOL (default system End-Of-Line)
  • +10 Get host machine CPUs info (overall amount of CPUS plus model and clock rate (in GHz) for each of them)
  • +6 Get home directory
  • +6 Get current system user name (Do not confuse with the username that is set when the application starts)
  • +6 Get CPU architecture for which Node.js binary has compiled

Hash calculation implemented properly

  • +20 Calculate hash for file

Compress and decompress operations

  • +20 Compress file (using Brotli algorithm)
  • +20 Decompress file (using Brotli algorithm)

Advanced Scope

  • +50 All files operations with reading/writing should be performed using Streams API
  • +20 Codebase is written in ESM modules instead of CommonJS
  • +20 Codebase is separated (at least 7 modules)

Tips

  • ls command show all files without dot on front, all files show with attribute ls -a

@webmakaka
Copy link

Привет!

Compress and decompress operations

Как правильно задать команду?

Screenshot from 2022-06-17 01-21-14

@webmakaka
Copy link

И команды для

  • Copy file
  • Move file

@Vitaljano
Copy link
Owner Author

compress path_to_file path_to_destination
compress folder/folder/filename.txt folder/folder/folder
decompress folder/folder/folder/main.br folder/folder/main.br
cp folder/folderfolder/copyfile.txt folder/folder/folder
Все по аналогий compress path_to_file path_to_destination

@webmakaka
Copy link

Отработали команды.
Оценку поправил.

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.

2 participants