Skip to content

Fix #8834: exclude paramters and variables from flag checks#8845

Merged
mhegazy merged 2 commits into
masterfrom
Fix8834
May 26, 2016
Merged

Fix #8834: exclude paramters and variables from flag checks#8845
mhegazy merged 2 commits into
masterfrom
Fix8834

Conversation

@mhegazy
Copy link
Copy Markdown
Contributor

@mhegazy mhegazy commented May 26, 2016

Fixes #8834

@mhegazy
Copy link
Copy Markdown
Contributor Author

mhegazy commented May 26, 2016

@RyanCavanaugh can you take a look.

Comment thread src/compiler/checker.ts Outdated
function areDeclarationFlagsIdentical(left: Declaration, right: Declaration) {
if ((left.kind === SyntaxKind.Parameter && right.kind === SyntaxKind.VariableDeclaration) ||
(left.kind === SyntaxKind.VariableDeclaration && right.kind === SyntaxKind.Parameter)) {
// Diffrences in optionality between paramters and variables are allowed.
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.

Spelling (Differences, parameters)

@RyanCavanaugh
Copy link
Copy Markdown
Member

👍

@mhegazy mhegazy merged commit b70d079 into master May 26, 2016
@mhegazy mhegazy deleted the Fix8834 branch May 26, 2016 21:35
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 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.

3 participants