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

♻️ REFACTOR: log wrappers for init-workspace.log #82

Closed
apolopena opened this issue Mar 30, 2021 · 0 comments
Closed

♻️ REFACTOR: log wrappers for init-workspace.log #82

apolopena opened this issue Mar 30, 2021 · 0 comments
Assignees
@apolopena
Labels
passed-dev-qa Optional state. Use this when QAing other peoples fixes in another branch.ready to be merged to main refactor

Comments

@apolopena
Copy link
Owner

Describe the bug

Currently every file that need to log to workspace-init.log has duplicated code for the log wrappers needed to log to that file.

Screenshots

image

Steps to reproduce

Open an file with a prefix of init- in the bash directory.

Expected behavior

logging to workspace-init.log should not require duplicating code.

Additional information

source a single file workspace-init-logger.sh into any file that need to log to workspace init.
@apolopena apolopena added the bug Something isn't working label Mar 30, 2021
@apolopena apolopena self-assigned this Mar 30, 2021
apolopena added a commit that referenced this issue Mar 30, 2021
@apolopena
♻️ REFACTOR: #82
d208572
apolopena added a commit that referenced this issue Mar 30, 2021
@apolopena
♻️ REFACTOR: #82 ready for dev-qa
4f6ee13
apolopena added a commit that referenced this issue Mar 30, 2021
@apolopena
🐛 FIX: #82
0a30cd0
apolopena added a commit that referenced this issue Mar 30, 2021
@apolopena
💡 UPDATE COMMENTS: #82, Add MIT license, unify headers in code files
2041896
apolopena added a commit that referenced this issue Mar 30, 2021
@apolopena
🐛 FIX: #82, change name to workspace-init-logger.sh
0bc6721
apolopena added a commit that referenced this issue Mar 30, 2021
@apolopena
🔀 MERGE: #82, change name to workspace-init-logger.sh
cd15c0a
apolopena added a commit that referenced this issue Mar 30, 2021
@apolopena
🐛 FIX: path to script for #82
7cc81be
apolopena added a commit that referenced this issue Mar 30, 2021
@apolopena
🐛 FIX: add missing && for #82
1581365
@apolopena apolopena added passed-dev-qa Optional state. Use this when QAing other peoples fixes in another branch.ready to be merged to main refactor and removed in-dev-qa bug Something isn't working passed-dev-qa Optional state. Use this when QAing other peoples fixes in another branch.ready to be merged to main refactor labels Mar 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@apolopena apolopena

Labels
passed-dev-qa Optional state. Use this when QAing other peoples fixes in another branch.ready to be merged to main refactor
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@apolopena