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.

Sign up for GitHub

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

Jump to bottom

Pub package layout conventions don't apply well with Polymer applications #13991

Closed
DartBot opened this issue Oct 10, 2013 · 7 comments
Closed

Pub package layout conventions don't apply well with Polymer applications #13991

DartBot opened this issue Oct 10, 2013 · 7 comments
Assignees
@Sfshaza
Labels
area-documentation Prefer using 'type-documentation' and a specific area label.

Comments

@DartBot
Copy link

DartBot commented Oct 10, 2013

This issue was originally filed by @butlermatt

The package layout conventions as described in Pub documentation[1] do not apply readily to Polymer applications. Where should component implementation files exist? Do implementation Polymer .html files belong in lib/src/ ? Do they belong in web/src/ ? what about their associated dart scripts/classes?

If they do belong in lib/src/ then this may also be BlockedOn Issue #12867
@kasperl
Copy link

kasperl commented Oct 14, 2013

Added Area-Polymer, Triaged labels.

@sigmundch
Copy link
Member

Bob/Nathan - what are your thoughts on this?

Removed Area-Polymer label.
Added Area-Pub label.

@munificent
Copy link
Member

Let's discuss this in person when Nathan is around. I'm not sure what we want to do here either.

@munificent
Copy link
Member

Removed Priority-Unassigned label.
Added Priority-Medium label.

@munificent
Copy link
Member

Marked this as being blocked by #16647.

@munificent
Copy link
Member

We spent some time talking about this and we've decided that we will officially support non-Dart files in "lib". I think the main missing piece now is updating the docs to reflect this.

Some notes on the discussion here: https://gist.github.com/munificent/8873127

cc @nex3.
Set owner to @Sfshaza.
Removed Area-Pub label.
Added Area-Documentation label.

@Sfshaza
Copy link

Sfshaza commented Mar 27, 2014

I've tweaked the "Assets and Transformers" page with regards to where assets can go:
https://www.dartlang.org/tools/pub/assets-and-transformers.html#where-to-put-assets

Added Fixed label.

@DartBot DartBot added Type-Defect area-documentation Prefer using 'type-documentation' and a specific area label. labels Mar 27, 2014
@DartBot DartBot mentioned this issue Jun 5, 2015
Closed
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sfshaza Sfshaza

Labels
area-documentation Prefer using 'type-documentation' and a specific area label.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@munificent @sigmundch @kasperl @Sfshaza @DartBot