Skip to content

fix: Minor updates to README#192

Merged
raj-jain-aws merged 3 commits intomainfrom
readme_refactor
Apr 20, 2026
Merged

fix: Minor updates to README#192
raj-jain-aws merged 3 commits intomainfrom
readme_refactor

Conversation

@raj-jain-aws
Copy link
Copy Markdown
Contributor

@raj-jain-aws raj-jain-aws commented Apr 19, 2026

Summary

Closes #193

Minor updates to README

Changes

  • Moved the "Experimental: AI-Assisted Setup" section lower in the document (after "Generated aidlc-docs/ Reference"), since it's supplementary content rather than a primary setup path. The installation prompt takes up 3% of context window before AI-DLC workflow even starts. Also, the instructions are written for *nix systems and not for Windows CMD and Windows Powershell.

  • Fixed ToC to match the new section order

  • Added missing "Generated aidlc-docs/ Reference" and "Contributing" entries to the ToC

  • Removed stale "Security" heading and replaced it with "Contributing" to match the section's actual content

  • Fixed -- → --- horizontal rule for proper markdown rendering

  • Added a note in CONTRIBUTING.md to test installation instructions on Mac, Windows CMD, and Windows PowerShell

User experience

Reverted installation sequence as before in terms of user experience

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • [*] I have reviewed the contributing guidelines
  • [*] I have performed a self-review of this change
  • [*] Changes have been tested
  • [*] Changes are documented

Test Plan

N/A

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 19, 2026
@raj-jain-aws raj-jain-aws changed the title Minor updates to README fix: Minor updates to README Apr 19, 2026
Comment thread CONTRIBUTING.md Outdated
Comment thread README.md Outdated
Copy link
Copy Markdown
Contributor

@leandrodamascena leandrodamascena left a comment

Choose a reason for hiding this comment

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

Hey @raj-jain-aws, thanks for the cleanup. I created #193 for traceability on this change.

I just left 2 comments and see if that make sense. Other than that, LGTM! The content is the same, you just reorganized.

raj-jain-aws and others added 2 commits April 19, 2026 15:36
Co-authored-by: Leandro Damascena <lcdama@amazon.pt>
Fixed header syntax
@raj-jain-aws
Copy link
Copy Markdown
Contributor Author

Hey @raj-jain-aws, thanks for the cleanup. I created #193 for traceability on this change.

I just left 2 comments and see if that make sense. Other than that, LGTM! The content is the same, you just reorganized.

Thanks @leandrodamascena for comments and opening an issue to capture this. Addressed both comments. Pls. review.

Copy link
Copy Markdown
Contributor

@leandrodamascena leandrodamascena left a comment

Choose a reason for hiding this comment

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

Thanks for addressing the feedback @raj-jain-aws. Approved.

@raj-jain-aws raj-jain-aws enabled auto-merge April 20, 2026 03:08
@raj-jain-aws raj-jain-aws added this pull request to the merge queue Apr 20, 2026
Merged via the queue into main with commit b7161b6 Apr 20, 2026
21 checks passed
@raj-jain-aws raj-jain-aws deleted the readme_refactor branch April 20, 2026 05:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs]: reorganize README and fix Table of Contents

3 participants