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.

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

Various workspaces - Removed usages of @backstage/backend-tasks #2356

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

awanlin
Copy link
Contributor

@awanlin awanlin commented Dec 24, 2024

Hey, I just made a Pull Request!

Removed usages of @backstage/backend-tasks as it has been deprecated for some time now. The changes in this PR are only to the example app that is not published and has no direct impact on the plugins themselves. This is also why there is no changeset.

✔️ 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)

@backstage-goalie
Copy link
Contributor

Changed Packages

Package Name Package Path Changeset Bump Current Version
backend workspaces/azure-devops/packages/backend none v0.0.12

@awanlin awanlin force-pushed the topic/remove-backend-tasks branch from 8f8c5b7 to fb5fa93 Compare December 24, 2024 20:34
@awanlin awanlin force-pushed the topic/remove-backend-tasks branch from fb5fa93 to 899be82 Compare January 7, 2025 18:32
@awanlin
Copy link
Contributor Author

awanlin commented Jan 7, 2025

Hi @JessicaJHee @christoph-jerolimov @deepan10 @vinzscam can you give this PR a look, please?

Copy link
Contributor

@JessicaJHee JessicaJHee left a comment

Choose a reason for hiding this comment

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

LGTM! Double-checked that all usages of @backstage/backend-tasks in the example apps have been removed.

Copy link
Member

@christoph-jerolimov christoph-jerolimov left a comment

Choose a reason for hiding this comment

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

Not involved in most of the plugins, but lgtm. :)

@vinzscam vinzscam merged commit 81cf366 into backstage:main Jan 14, 2025
42 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