Skip to content

typo: propelml -> ry#10

Merged
ry merged 2 commits into
denoland:masterfrom
qti3e:q/readme
May 29, 2018
Merged

typo: propelml -> ry#10
ry merged 2 commits into
denoland:masterfrom
qti3e:q/readme

Conversation

@qti3e

@qti3e qti3e commented May 29, 2018

Copy link
Copy Markdown
Contributor

No description provided.

@ry ry left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ry
ry merged commit 247932b into denoland:master May 29, 2018
@qti3e
qti3e deleted the q/readme branch May 30, 2018 13:11
bartlomieju added a commit that referenced this pull request Jul 9, 2026
- extract_tarball_gz: skip symlink/hardlink tar entries (path-traversal via a
  symlink to an outside dir + a file nested under it, on arbitrary registry/@jsr
  content).
- Rebase the user's compilerOptions.paths + baseUrl onto .deno/ (the generated
  tsconfig lives one level down); they were copied verbatim, so `~/*`->`./src/*`
  resolved to .deno/src/* and broke every import through a user alias. Dropped
  paths/baseUrl from passthrough; added rebase_onto_deno_dir + test.
- sync_types_command: check node_modules/.deno components on the path RELATIVE to
  the project root, so an ancestor dir named node_modules/.deno no longer filters
  every root away (empty graph, no error).
- ensure_root_tsconfig: edit the existing tsconfig by byte-range splice via the
  jsonc AST instead of re-serializing, preserving the user's comments and
  formatting (previously silently stripped on the first sync).
- strip_top_level_type_decls: bound the leading-comment back-pop to contiguous
  comment lines (an unbounded 'pop until /**' could delete earlier decls), and
  make the type-alias skip track bracket depth so an interior ';' doesn't end it
  early.

Deferred (noted on the PR): resolution flags incl --allow-import (#3),
skipped-remote policy (#4), import-map scopes (#8), @types/node via #35889 (#10).
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.

2 participants