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

data/Language/Flow.yaml: Add Flow definition #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

data/Language/Flow.yaml: Add Flow definition #91

wants to merge 1 commit into from
+78 −0

Conversation

utkarsh2102
Copy link
Member

This adds definition of Flow, a static type checker for JS.

Closes #51

@jayvdb
Copy link
Member

jayvdb commented Dec 5, 2018

@gitmate-bot rebase

@jayvdb
Copy link
Member

jayvdb commented Dec 5, 2018

Hey! I'm GitMate.io! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost permanently ⚠️

@jayvdb
Copy link
Member

jayvdb commented Dec 5, 2018

Automated rebase with GitMate.io was successful! 🎉

jayvdb
jayvdb requested changes Dec 5, 2018
View reviewed changes
Copy link
Member

@jayvdb jayvdb left a comment

Choose a reason for hiding this comment

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

this language has no keywords?

no boolean or null literals?

data/Language/Flow.yaml Outdated Show resolved Hide resolved
data/Language/Flow.yaml Outdated Show resolved Hide resolved
@utkarsh2102
Copy link
Member Author

utkarsh2102 commented Dec 5, 2018 •
edited
Loading

this language has no keywords?

Flow is just a static type checker for JS. So thought of skipping the same.
Should I add it?

@jayvdb
Copy link
Member

jayvdb commented Dec 5, 2018

Flow is just a static type checker for JS.

coAST is about creating models to describe stuff. Not about omitting information.

@utkarsh2102
Copy link
Member Author

Flow is just a static type checker for JS.

coAST is about creating models to describe stuff. Not about omitting information.

Duly noted.
Will add them.
Do reply to the comments inline, too.

@jayvdb
Copy link
Member

jayvdb commented Dec 5, 2018

I have no intention of giving more feedback at this stage, as you have not yet attempted to model the language.

@utkarsh2102
Copy link
Member Author

Will do, thanks.

@jayvdb
Copy link
Member

jayvdb commented Dec 12, 2018

Comment on 6a22f54, file data/Language/VisualBasic.yaml, line 5.

line too long (82 > 80 characters) (line-length)

Origin: YAMLLintBear, Section: yaml.

pythongiant
pythongiant approved these changes Dec 12, 2018
View reviewed changes
Copy link
Member

@pythongiant pythongiant left a comment

Choose a reason for hiding this comment

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

LGTM. nice stuff

@utkarsh2102
Copy link
Member Author

Comment on 6a22f54, file data/Language/VisualBasic.yaml, line 5.

line too long (82 > 80 characters) (line-length)

Origin: YAMLLintBear, Section: yaml.

This is not related to this PR. Moreover, this has been fixed already.

@li-boxuan
Copy link
Member

Needs rebase :)

@utkarsh2102
Copy link
Member Author

Needs rebase :)

Rebased. How does it look now?

li-boxuan
li-boxuan requested changes Dec 13, 2018
View reviewed changes
data/Language/Flow.yaml Outdated Show resolved Hide resolved
@utkarsh2102
Copy link
Member Author

@li-boxuan, done as asked :D
Could you take a look now?

@li-boxuan
Copy link
Member

Netlify is failing due to other reason now :)

@utkarsh2102
Copy link
Member Author

Netlify is failing due to other reason now :)

My bad. Resolved, @li-boxuan.
Let me know if there's any other issue in this PR.

@utkarsh2102
Copy link
Member Author

Rebased, too.

Naveenaidu
Naveenaidu approved these changes Dec 16, 2018
View reviewed changes
Copy link
Member

@Naveenaidu Naveenaidu left a comment

Choose a reason for hiding this comment

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

👍 LGTM

li-boxuan
li-boxuan reviewed Dec 16, 2018
View reviewed changes
data/Language/Flow.yaml Outdated Show resolved Hide resolved
@pythongiant
Copy link
Member

I was about to point out what @li-boxuan did, but it looks good now.
The new modeling fixes this up.

ksdme
ksdme reviewed Dec 31, 2018
View reviewed changes
Copy link
Member

@ksdme ksdme left a comment

Choose a reason for hiding this comment

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

This could use a logo. https://commons.wikimedia.org/wiki/File:Flow_Static_Typechecker_for_JavaScript.svg

data/Language/Flow.yaml Outdated Show resolved Hide resolved
Naveenaidu
Naveenaidu approved these changes Jan 30, 2019
View reviewed changes
Copy link
Member

@Naveenaidu Naveenaidu left a comment •
edited
Loading

Choose a reason for hiding this comment

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

👍 LGTM

KVGarg
KVGarg approved these changes Feb 9, 2019
View reviewed changes
Copy link

@KVGarg KVGarg left a comment

Choose a reason for hiding this comment

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

LGTM 💯 👍

@utkarsh2102
data/Language/Flow.yaml: Add Flow definition
a39a062 
This adds definition of Flow, a static type checker for JS.

Closes coala#51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@li-boxuan li-boxuan li-boxuan requested changes

@jayvdb jayvdb jayvdb requested changes

@ksdme ksdme ksdme approved these changes

@pythongiant pythongiant pythongiant approved these changes

@shashank-b shashank-b shashank-b approved these changes

@frextrite frextrite frextrite approved these changes

@Naveenaidu Naveenaidu Naveenaidu approved these changes

@rajgoesout rajgoesout rajgoesout approved these changes

@KVGarg KVGarg KVGarg approved these changes

Assignees

@utkarsh2102 utkarsh2102

Labels
difficulty/low process/pending review size/S
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@utkarsh2102 @jayvdb @li-boxuan @pythongiant @ksdme @shashank-b @frextrite @Naveenaidu @rajgoesout @KVGarg