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.

Sign up for GitHub

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

Jump to bottom

Error Handling should be improved #37

Open
2 of 3 tasks
anilhelvaci opened this issue Jul 27, 2021 · 0 comments
Open
2 of 3 tasks

Error Handling should be improved #37

anilhelvaci opened this issue Jul 27, 2021 · 0 comments
Assignees
@anilhelvaci
Labels
feature New feature or request

Comments

@anilhelvaci
Copy link
Collaborator

anilhelvaci commented Jul 27, 2021 •
edited
Loading

  • Better custom error classes are needed
  • Error handler function should contain more logic
  • Integrate winston for logging
@anilhelvaci anilhelvaci added the feature New feature or request label Jul 27, 2021
@anilhelvaci anilhelvaci self-assigned this Jul 27, 2021
anilhelvaci added a commit that referenced this issue Jul 27, 2021
@anilhelvaci
#37 First look at the error handling
405ba51
anilhelvaci added a commit that referenced this issue Jul 29, 2021
@anilhelvaci
#37 Wrapped errors are applied to refactored installation module
e842e4c
anilhelvaci added a commit that referenced this issue Jul 29, 2021
@anilhelvaci
#37 Caught errors are directed to the ErrorHandler from controllers
cdaa49c
anilhelvaci added a commit that referenced this issue Jul 30, 2021
@anilhelvaci
#37 #38 Context module is refactored and error handling is applied.
fedfca7
anilhelvaci added a commit that referenced this issue Jul 30, 2021
@anilhelvaci
#37 Errors regarding to file system are wrapped and thrown.
3e360ca
anilhelvaci added a commit that referenced this issue Jul 31, 2021
@anilhelvaci
#37 GenericError class is created and applied to controllers.
0cb3d93
anilhelvaci added a commit that referenced this issue Aug 2, 2021
@anilhelvaci
#37 Integration of winston is started
5093ae3
anilhelvaci added a commit that referenced this issue Aug 4, 2021
@anilhelvaci
#37 Debug levels implemented, 'info' level is done, 'debug' level sta…
08a3991 
…rted.
anilhelvaci added a commit that referenced this issue Aug 5, 2021
@anilhelvaci
#37 Debug level continued
1c481af
anilhelvaci added a commit that referenced this issue Aug 6, 2021
@anilhelvaci
#37 debug level completed, runBasicCmd method is running async and …
6eb7a8c 
…fetching stdout to print to log files
anilhelvaci added a commit that referenced this issue Aug 9, 2021
@anilhelvaci
#37 env variable validation added, some improvements made to the outp…
0893139 
…ut log file.
@anilhelvaci anilhelvaci mentioned this issue Aug 9, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anilhelvaci anilhelvaci

Labels
feature New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@anilhelvaci