Skip to content

Conversation

@sockmaster27
Copy link
Contributor

@sockmaster27 sockmaster27 commented Aug 23, 2024

This adds the Flix programming language.

Flix started as a research project, but is today a fully-featured programming language. It has focused on practical tooling for making the language widely usable, which has helped spark its growing open-source community.

Checklist:

@sockmaster27 sockmaster27 requested a review from a team as a code owner August 23, 2024 15:27
@sockmaster27
Copy link
Contributor Author

@lildude
I can see that you've added the "Pending Popularity" tag even though it looks like the extension has over 2k files.
Would you mind sharing what filters you're using or if you're judging by other criteria? It would be helpful to know :)

@lildude
Copy link
Member

lildude commented Aug 30, 2024

Sure. https://github.com/search?type=code&q=NOT+is%3Afork+path%3A*.flix+NOT+user%3Aflix

I always exclude the language "owner" and other "big hitters" as they often disproportionately affect the results, which is clearly the case here.

@sockmaster27
Copy link
Contributor Author

That makes sense, thanks!

@magnus-madsen
Copy link

We have had some recent uptake and it looks like we now satisfy the criteria-- although the search is a bit wonky.

@saviorand
Copy link

@lildude looking at the search now, I think there are more than 2k files with the filters. Could you please take a look to see if it satisfies the popularity requirement now?

@lildude
Copy link
Member

lildude commented Jul 8, 2025

I only re-review language popularity when I make a new release which is approx every 3 months. I'll reassess then.

Copy link
Member

@lildude lildude left a comment

Choose a reason for hiding this comment

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

Usage looks good, however you've got merge conflicts. Please resolve the merge conflicts.

@sockmaster27
Copy link
Contributor Author

@lildude Should be ready now

Copy link
Member

@lildude lildude left a comment

Choose a reason for hiding this comment

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

Please revert the update to the vendor/grammars/language-xbase submodule. We only update submodules when we make a release.

@saviorand
Copy link

@lildude should be good now?

Copy link
Member

@lildude lildude left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

Important

The changes in this PR will not appear on GitHub until the next release has been made and deployed. See here for more details.

@lildude lildude added this pull request to the merge queue Sep 15, 2025
Merged via the queue into github-linguist:main with commit 168940d Sep 15, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants