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

Code conformance for aws-apps plugin #132

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fleveillee
Copy link

What does this PR do?

🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

We have corrected some minor TypeScript syntax mistakes and aligned the code using prettier, as per the Backstage community's recommended practices.

Motivation

The main objective is to decouple AWS Harmonix plugins with the opa-platform provisioning requirements and enable to run Harmonix in vanilla Backstage app (see: https://github.com/alithya-oss/backstage-plugins/tree/main/workspaces/aws ).

  • Added API report yarn build:api-reports:only
  • Applied Prettier code formatting yarn prettier write
  • Fix repo yarn fix --publish

For Moderators

  • Compile, build, tests successful before merge?

Additional Notes

Relates to issues #116

@fleveillee fleveillee marked this pull request as ready for review November 11, 2024 14:49
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.

1 participant