Skip to content

Fix: root README linking and domain badges#1625

Merged
TheOrangePuff merged 2 commits intoaligent:mainfrom
finnholland:fix/readme-linking
Mar 27, 2026
Merged

Fix: root README linking and domain badges#1625
TheOrangePuff merged 2 commits intoaligent:mainfrom
finnholland:fix/readme-linking

Conversation

@finnholland
Copy link
Copy Markdown
Contributor

Description of the proposed changes

  • fixed links in readme pointing to /packages/<package> not /packages/constructs/<package> now the root readme will redirect
  • Also updated the yarn install step

Here's my fork with the links for example: readme-linking

Notes to reviewers

🛈 When you've finished leaving feedback, please add a final comment to the PR tagging the author, letting them know that you have finished leaving feedback

@finnholland finnholland requested a review from a team as a code owner March 11, 2026 01:46
@finnholland
Copy link
Copy Markdown
Contributor Author

Looks like the check for packages is run against /packages which would list cdk-aspects and constructs should that be lowered into packages/constructs?

@TheOrangePuff
Copy link
Copy Markdown
Member

@finnholland I think the readme update workflow has been fixed, you might need to rebase or merge main into this PR. But then should be good to merge 😄

will also want minor for readme changes but thats in release.
@finnholland
Copy link
Copy Markdown
Contributor Author

Well that fork was way out of sync lol... that's looking a bit better now!

@TheOrangePuff TheOrangePuff merged commit 44d40ce into aligent:main Mar 27, 2026
8 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.

2 participants