Skip to content

OpenAPI Generation for backend-common through tsoa#15038

Closed
sennyeya wants to merge 1 commit into
backstage:masterfrom
sennyeya:Swagger
Closed

OpenAPI Generation for backend-common through tsoa#15038
sennyeya wants to merge 1 commit into
backstage:masterfrom
sennyeya:Swagger

Conversation

@sennyeya

@sennyeya sennyeya commented Dec 6, 2022

Copy link
Copy Markdown
Contributor

Signed-off-by: Aramis Sennyey sennyeya@amazon.com

Hey, I just made a Pull Request!

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

Signed-off-by: Aramis Sennyey <sennyeya@amazon.com>
@github-actions github-actions Bot added the area:catalog Related to the Catalog Project Area label Dec 6, 2022
@github-actions

github-actions Bot commented Dec 6, 2022

Copy link
Copy Markdown
Contributor

Missing Changesets

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

  • @backstage/backend-common
  • @backstage/cli
  • @backstage/plugin-catalog-backend

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

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage/backend-common packages/backend-common none v0.17.0-next.1
@backstage/cli packages/cli none v0.21.2-next.1
@backstage/plugin-catalog-backend plugins/catalog-backend none v1.6.0-next.1

"cors": "^2.8.5",
"dockerode": "^3.3.1",
"express": "^4.17.1",
"express": "^4.18.2",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do not update this one. It's kept the same across the entire repo.

tsx: !isBackend,
dynamicImport: true,
decorators: true,
decoratorsBeforeExport: true,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We are not ready to put in decorator support; this particular change will unfortunately be rejected. We want to postpone until we know more where the TS ecosystem lands in regards to this, and would need VERY good reason to add it at this point.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

👍 , makes sense. Similarly, where does the project land on IoC (Inversion of Control), https://github.com/backstage/backstage/pull/15038/files#diff-bcf7cd81750d2e438f7dc4b53af26cf084dc7323499cb498132ad9749ce959e3R1?

@github-actions

Copy link
Copy Markdown
Contributor

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!

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 stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants