-
Notifications
You must be signed in to change notification settings - Fork 84
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
feat: added logging service #860
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mdesmet
reviewed
Jan 30, 2024
src/dbt_client/dbtTerminal.ts
Outdated
} | ||
} | ||
|
||
error(e: CustomException, sendTelemetry = false) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
by default sendTelemetry of errors.
49aad78
to
b386fe4
Compare
b386fe4
to
339c43f
Compare
mdesmet
approved these changes
Feb 1, 2024
svipulrao98
pushed a commit
that referenced
this pull request
Feb 13, 2024
author anandgupta42 <anand@altimate.ai> 1706765937 -0800 committer Vipul Rao <svipulrao98@gmail.com> 1707810351 +0530 bumping version for pre-release 0.31.1 chore: create colors css variables in webviewpanel for core theme (#876) * chore: create colors theme * chore: add datapilot, doc editor stories * fix: gitpod integration bumping version for release 0.31.2 extension basic changes - env variables set - profiles - target fixes: - call the target from the py bridge - remove util function remove unused imports remove needless commented code make sure we return the abs path remove testing changes minor fixes: - project dir in the target path too - dont use cwd set the env vars in the settings json file. path, stricyly relative to project dir don't do path join if the target is outside of the project remove the env setting changes remove unnecessary initialization minor fixes Remove imports Rename vars fix: feedback link (#878) feat: lineage cancellation (#863) * chore: code refactor * feat: combine start and end progress * WIP * feat: cancelling some request * chore: remove unused cache code * chore: code refactor * fix: cancellation and no lineage * chore: code refactor * feat: prevent actions when cll in progress * WIP * feat: cancel on reset, show old ui * feat: remove ctxId from host * feat: remove ctxId from webview * feat: change name to CllContext * chore: code refactor * chore: code cleanup * feat: added telemtry events * WIP * chore: refactor no lineage msg to info notif * WIP * chore: code refactor * WIP Revert "extension basic changes" (#879) This reverts commit ed746eb. bumping version for release 0.31.3 show the analysis options by default. (#886) feat: added logging service (#860) * feat: WIP * feat: added error handle in docsEditPanel * chore: removed unused imports fix: use float instead of int in query results to avoid overflow (#889) * fix: use float instead of int in query results to avoid overflow * Update main.js Feat/target path profiles dir (#880) * Revert "Revert "extension basic changes" (#879)" This reverts commit 1ca2d26. * Revert "Revert "extension basic changes" (#879)" This reverts commit 1ca2d26. bumping version for release 0.31.4 disabling showing of analysis panel bumping version for release 0.31.5 feat: lineage layout improvements (#883) * fix: ordering within level based on dfs WIP chore: code refactor WIP chore: code refactor fix: overlap issue WIP chore: code refactor chore: code refactor chore: code refactor WIP WIP WIP WIP WIP * chore: code cleanup * WIP * feat: added comments Fix profiles dir bump version to 0.31.6 Fix package filtering fix: query panel horizontal scrolling issue (#899) feat: auto expand lineage (#887) * chore: code refactor chore: code refactor chore: code refactor WIP WIP chore: code refactor chore: code refactor fix: remove reset table selection WIP WIP WIP WIP WIP WIP WIP chore: code refactor chore: code refactor WIP chore: code refactor chore: code refactor WIP WIP WIP WIP fix: styling WIP chore: code refactor chore: code refactor chore: code refactor fix: styling fix: styling chore: code refactor chore: code refactor chore: code refactor feat: new approach fix: new approach expand fix: new approach node count WIP feat: added range feat: resetting state on selection change chore: code refactor chore: code refactor WIP * feat: enable auto expansion * fix: text * WIP * fix: reset highlighting * WIP * feat: set min range * WIP * WIP * fix: widths * fix: layout algo * chore: code refactor * fix: layout algo * chore: code refactor * chore: code refactor * fix: see more related stuff * fix: styling * fix: styling * fix: popover click outside * chore: code refactor * WIP * fix: popover * WIP * chore: code refactor * fix: issues * fix: styling * WIP * WIP * WIP * WIP * fix: popover * Revert "fix: popover click outside" This reverts commit 9d833ec. * WIP * WIP * WIP bumping version for release 0.32.0 improve: use faker and factory.ts for better test data generation (#893) fallback to read profiles if python fails use vanilla YAML instead of js-yaml. check path exists before returning update var name update var name
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WIP
fix: logging service
WIP
WIP
feat: combined logging and terminal
feat: added strip ansi
feat: optional telemetry
WIP
fix: as per comments
WIP
Overview
Checklist
README.md
updated and added information about my change