Skip to content

Make boolean compiler options in harness true only if they say 'true'#3093

Merged
JsonFreeman merged 1 commit intorelease-1.5from
harnessCompilerOptions
May 8, 2015
Merged

Make boolean compiler options in harness true only if they say 'true'#3093
JsonFreeman merged 1 commit intorelease-1.5from
harnessCompilerOptions

Conversation

@JsonFreeman
Copy link
Contributor

Today, if you say

//@declaration: false

in a test, it will emit declarations! The harness just checks for the presence or absence of these options. It does not respect false.

Copy link
Member

Choose a reason for hiding this comment

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

You can consider just performing this once in a variable prior to the switch, and then using that variable in the appropriate locations.

@DanielRosenwasser
Copy link
Member

👍

JsonFreeman added a commit that referenced this pull request May 8, 2015
Make boolean compiler options in harness true only if they say 'true'
@JsonFreeman JsonFreeman merged commit 65b7f53 into release-1.5 May 8, 2015
@vladima
Copy link
Contributor

vladima commented May 8, 2015

👍

@microsoft microsoft locked and limited conversation to collaborators Jun 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants