-
Notifications
You must be signed in to change notification settings - Fork 3
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
Type system support #129
Type system support #129
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #129 +/- ##
=======================================
Coverage 83.11% 83.11%
=======================================
Files 11 11
Lines 308 308
Branches 80 80
=======================================
Hits 256 256
Misses 52 52
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
@fdodino se habia movido el log y el time measurement del buildeo del environment asi que al mergearlo con el nuevo logger lo deje asi
const message = `✘ Failed to build environment: ${error}` | ||
logger.error({ | ||
level: 'error', | ||
files: files.map(file => file.name), | ||
message, | ||
}) | ||
throw error |
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.
Lo puse aca, que es donde se buildea el environment
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.
Piola!
message, | ||
}) | ||
connection.console.error(message) | ||
connection.console.error(`✘ Failed to rebuild document: ${e}`) |
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.
Como ahora el buildEnvironment
hace el log aca deje solo el mostrar el error al usuario
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.
🍾
const message = `✘ Failed to build environment: ${error}` | ||
logger.error({ | ||
level: 'error', | ||
files: files.map(file => file.name), | ||
message, | ||
}) | ||
throw error |
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.
Piola!
test('hover field', async () => { | ||
await testHover( | ||
hoverURI, | ||
new Position(1, 8), | ||
new Hover( | ||
[ | ||
new MarkdownString('\n```text\nField: Number\n```\n'), | ||
new MarkdownString('\n```wollok\nconst x = 2\n```\n'), | ||
], | ||
new Range(new Position(1, 2), new Position(2, 0)) | ||
) | ||
) | ||
}) | ||
}) |
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.
🏅
RnR 🚀🚀🚀 |
Resuelve item de uqbar-project/wollok-ts#187
Wollok? Un tipazo
Depende de uqbar-project/wollok-ts#203