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

Handle combinations of rootdir and outdir when calculating paths #24941

Merged
merged 1 commit into from
Jun 14, 2018

Conversation

weswigham
Copy link
Member

By always calculating and using absolute paths when looking for symlinks.

Fixes this comment.

Followup to #24874.

@weswigham weswigham requested review from rbuckton and mhegazy June 14, 2018 00:45
@weswigham weswigham merged commit a770688 into microsoft:master Jun 14, 2018
@weswigham weswigham deleted the handle-rootdir-and-outdir branch June 14, 2018 18:07
@alan-agius4
Copy link
Contributor

alan-agius4 commented Jul 30, 2018

In which version will this be out? As with TypeScript 2.9 this is still any issue.

@mhegazy
Copy link
Contributor

mhegazy commented Jul 30, 2018

2.9.2. please file a new a ticket and give us more details to be able to diagnose.

@alan-agius4
Copy link
Contributor

I have tried TS3 and the issue is fixed. Will either try to find which commit solved it, if I find some time in the coming days. If they is a chance for a patch, else I’ll wait for V3.

Thanks

@alan-agius4
Copy link
Contributor

alan-agius4 commented Jul 30, 2018 via email

@raymondfeng
Copy link

@weswigham The original issue is mostly fixed. But we just ran into a variant of the problem now. See loopbackio/loopback-next#1643.

In this case, MetadataAccessor is exported from @loopback/metadata and re-exported by @loopback/context. Please note that @loopback/repository-json-schema module imports MetadataAccessor from @loopback/context. It leads to incorrectly generated code:

import { MetadataAccessor } from '@loopback/context';
import { JSONSchema6 as JSONSchema } from 'json-schema';
/**
 * Metadata key used to set or retrieve repository JSON Schema
 */
export declare const JSON_SCHEMA_KEY: MetadataAccessor<JSONSchema, import("../../context/node_modules/@loopback/metadata/src/types").DecoratorType>;

Switching the import to @loopback/metadata seems to fix the problem.

@raymondfeng
Copy link

@weswigham Any update? Should we open a new issue to track this?

@weswigham
Copy link
Member Author

New issue would make it easier to track.

@raymondfeng
Copy link

@weswigham See #26985

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.

4 participants