Skip to content

Introducing the Linguist plugin#14718

Merged
freben merged 7 commits intobackstage:masterfrom
ahhhndre:topic/introduce-linguist-plugin
Feb 14, 2023
Merged

Introducing the Linguist plugin#14718
freben merged 7 commits intobackstage:masterfrom
ahhhndre:topic/introduce-linguist-plugin

Conversation

@ahhhndre
Copy link
Contributor

Hey, I just made a Pull Request!

Introducing the Linguist plugin: get a nice breakdown of all the languages that make up an Entity in the Catalog! This is great for those SCMs that don't offer this feature like Azure DevOps or BitBucket, but it still works with the others as well. Closes #9733

linguist-with-data

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)
  • All your commits have a Signed-off-by line in the message. (more info)

@github-actions github-actions bot added the area:catalog Related to the Catalog Project Area label Nov 20, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Nov 20, 2022

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @backstage/catalog-model

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
example-app packages/app none v0.2.78-next.3
example-backend packages/backend none v0.2.78-next.3
@backstage/catalog-model packages/catalog-model none v1.1.4-next.1
@backstage/plugin-linguist-backend plugins/linguist-backend minor v0.0.0
@backstage/plugin-linguist-common plugins/linguist-common minor v0.0.0
@backstage/plugin-linguist plugins/linguist minor v0.0.0

@ahhhndre
Copy link
Contributor Author

Last Christmas I created a plugin
but it took me a year to open source it

@freben 😉

Copy link
Contributor Author

@ahhhndre ahhhndre left a comment

Choose a reason for hiding this comment

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

Somethings of note for when you start to review this

@github-actions github-actions bot added area:documentation Improvements or additions to documentation area:microsite Changes to backstage.io area:scaffolder Everything and all things related to the scaffolder project area search area:techdocs Related to the TechDocs Project Area labels Nov 20, 2022
@ahhhndre ahhhndre force-pushed the topic/introduce-linguist-plugin branch from 1572dc7 to 7354b23 Compare November 20, 2022 19:39
@github-actions github-actions bot removed area:scaffolder Everything and all things related to the scaffolder project area area:documentation Improvements or additions to documentation area:microsite Changes to backstage.io search area:techdocs Related to the TechDocs Project Area labels Nov 20, 2022
@backstage-goalie
Copy link
Contributor

Thanks for the contribution!
All commits need to be DCO signed before they are reviewed. Please refer to the the DCO section in CONTRIBUTING.md or the DCO status for more info.

@ahhhndre ahhhndre force-pushed the topic/introduce-linguist-plugin branch from 7354b23 to eea33cc Compare November 21, 2022 23:26
@ahhhndre
Copy link
Contributor Author

ahhhndre commented Nov 22, 2022

@freben, could use some help. Somehow when I rebased I got 3 commits missed into my branch and I'm not sure how best to get them out. Any suggestions? My git-fu is failing me

@ahhhndre ahhhndre force-pushed the topic/introduce-linguist-plugin branch from eea33cc to ddc272f Compare November 22, 2022 21:21
@GLundh
Copy link
Contributor

GLundh commented Nov 23, 2022

This is such a great plugin. Thanks for contributing this. Looking forward to have it merged :)

@freben freben force-pushed the topic/introduce-linguist-plugin branch from ddc272f to a1ca9da Compare November 24, 2022 13:02
Copy link
Member

@freben freben left a comment

Choose a reason for hiding this comment

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

Hi, i know this is a draft, but here are some random comments!

@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2022

This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution!

@github-actions github-actions bot added the stale label Dec 1, 2022
@ahhhndre ahhhndre force-pushed the topic/introduce-linguist-plugin branch from a1ca9da to 97154c7 Compare December 4, 2022 00:59
@ahhhndre ahhhndre requested review from a team and backstage-service as code owners January 7, 2023 22:28
@ahhhndre ahhhndre force-pushed the topic/introduce-linguist-plugin branch from 68fe14e to 71ff35d Compare January 12, 2023 19:40
@ahhhndre
Copy link
Contributor Author

Alright, I'm still working through the comments so you can hold off on this for a while still. Making progress though 👍

@ahhhndre ahhhndre force-pushed the topic/introduce-linguist-plugin branch 2 times, most recently from b8e502a to 2e9d21d Compare January 22, 2023 19:11
@github-actions
Copy link
Contributor

github-actions bot commented Feb 1, 2023

This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution!

@github-actions github-actions bot added the stale label Feb 1, 2023
@freben freben removed the stale label Feb 3, 2023
@freben
Copy link
Member

freben commented Feb 3, 2023

@ahhhndre just checking in, is it at the point where just a rebase and merge and then tweak, would be desirable?

@ahhhndre ahhhndre force-pushed the topic/introduce-linguist-plugin branch from 2e9d21d to 24fa0c8 Compare February 4, 2023 19:29
@ahhhndre ahhhndre requested a review from a team as a code owner February 4, 2023 19:29
@ahhhndre ahhhndre requested a review from benjdlambert February 4, 2023 19:29
@ahhhndre ahhhndre force-pushed the topic/introduce-linguist-plugin branch from 0274090 to 0bafb08 Compare February 4, 2023 19:30
Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>

Removed refresh and permissions

Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>

Small database refactor and improved naming

Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>

Refactor stringifyEntityRef usage

Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>

Fixed up yarn.lock

Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>
Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>
Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>
Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>
Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>
@ahhhndre ahhhndre force-pushed the topic/introduce-linguist-plugin branch from a26837b to 672eade Compare February 11, 2023 21:00
Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>
Signed-off-by: Andre Wanlin <andrewanlin@gmail.com>
@ahhhndre
Copy link
Contributor Author

Alright, at this stage I think it's good for approval and getting merged in, but if you want to do another review that works for me as well @freben

Copy link
Member

@freben freben left a comment

Choose a reason for hiding this comment

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

Let's go!

@freben freben merged commit bba9cab into backstage:master Feb 14, 2023
@github-actions
Copy link
Contributor

Thank you for contributing to Backstage! The changes in this pull request will be part of the 1.11.0 release, scheduled for Tue, 14 Feb 2023.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:catalog Related to the Catalog Project Area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Plugin] Linguist (Languages)

6 participants