FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Show the dominant language of a tip4commit repo by joshumax · Pull Request #107 · tip4commit/tip4commit · GitHub

Repository navigation

Show the dominant language of a tip4commit repo - #107

Open
joshumax wants to merge 2 commits into
tip4commit:masterfrom
joshumax:master
Open

joshumax wants to merge 2 commits into
tip4commit:masterfrom
joshumax:master

Conversation

Copy link
Copy Markdown

This commit allows users to quickly see what language their favorite tip4commit repo is written in.

Copy link
Copy Markdown
Author

@AlexandrZ Thoughts?

Copy link
Copy Markdown
Member

I would prefer to move it somewhere else and leave repo names only in the first column. Maybe it's better to show a repo language after its name or in a separate column.

Now it's possible to search by a language. Maybe we don't need to show languages in the list, but add a filter in the header with the list of available languages.

Copy link
Copy Markdown
Author

@AlexandrZ I can certainly try moving the tags around to see what would look best...As for the second choice of adding a filter, that might be a bit tougher...since statically defining a list of languages to filter by would be quite odd. One way to do this might be to query all of the tip4commit repos at once and insert each unique language to a dynamic filter list, but wouldn't that just create a whole bunch of unneeded backend traffic through the github api? Would there be a better way to do this?

Copy link
Copy Markdown
Member

It's easy to get the list: Project.uniq.pluck(:language)

Copy link
Copy Markdown
Contributor

@joshumax are you continuing this?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL