Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

Initial parser port.#1

Merged
pq merged 5 commits into
masterfrom
initial_port
May 14, 2015
Merged

Initial parser port.#1
pq merged 5 commits into
masterfrom
initial_port

Conversation

@pq

@pq pq commented May 13, 2015

Copy link
Copy Markdown
Contributor

An initial port of Lasse's parser implementation.

@lrhn , @hterkelsen : PTAL!

FYI: @sethladd.

Comment thread lib/packagemap.dart Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

for consistency, use int

@pq

pq commented May 13, 2015

Copy link
Copy Markdown
Contributor Author

Needless to say backing out path normalization is not a long term fix. We could change our SDK version constraint obviously but then that would make the library less client-friendly. Anyway, something to consider.

pq referenced this pull request in googlearchive/analyzer_cli May 13, 2015
@pq

pq commented May 14, 2015

Copy link
Copy Markdown
Contributor Author

@sethladd PTAL?

@sethladd

Copy link
Copy Markdown
Contributor

I think it's OK to require 1.11 for this package.

Otherwise, LGTM

@pq

pq commented May 14, 2015

Copy link
Copy Markdown
Contributor Author

I think it's OK to require 1.11 for this package.

I opened dart-lang/tools#1501 to keep track of this. In the short term I think making it play nice with <1.11 will make non-bleeding-edgers happy.

pq added a commit that referenced this pull request May 14, 2015
@pq pq merged commit 0a78197 into master May 14, 2015
@pq pq deleted the initial_port branch May 14, 2015 18:00
mosuem pushed a commit to dart-lang/tools that referenced this pull request Dec 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants