Skip to content
This repository has been archived by the owner on Feb 5, 2019. It is now read-only.

Allow use conditional directives at least within the section of the import/export directives #10

Closed
mezoni opened this issue Mar 5, 2015 · 4 comments

Comments

@mezoni
Copy link

mezoni commented Mar 5, 2015

// Directives import/export
// Here allowed conditional directives

// Declarations
// Here not allowed conditional directives
library name

#if env_var == constant expression
// Directives import/export
#elif env_var == constant expression
// Directives import/export
#else
// Directives import/export
#endif
library name

#if dart_platform == "server" || dart_platform == "browser"
import "common.dart";
#else
#error Unsupported platform
#endif

#if dart_platform == "server"
import "server.dart";
#elif  dart_platform == "browser"
import "browser.dart";
#else
#error Unsupported platform
#endif
@a14n
Copy link

a14n commented Mar 5, 2015

It looks like #6.

@mezoni
Copy link
Author

mezoni commented Mar 5, 2015

This is issue for it but I cannot understand "why was chosen github issue per proposal?"
I not expected that was looks like the new proposal.
I.e. github issue means a proposal?
But where discuss about the proposal and place issues for proposal?
In the body of issue (proposal)?

@jmesserly
Copy link
Contributor

please file this issue on the tracker for the proposal: https://github.com/lrhn/dep-configured-imports/issues

@munificent
Copy link
Contributor

I.e. github issue means a proposal?

The GitHub issue here tracks the status of a proposal. Each proposal has its own repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

4 participants