Skip to content

Clarify frontend installation and test coverage instructions - #90

Merged
Kaap10 merged 1 commit into
PicadoLabs:mainfrom
dhruvatr:patch-1
Oct 29, 2026
Merged

Kaap10 merged 1 commit into
PicadoLabs:mainfrom
dhruvatr:patch-1

Conversation

@dhruvatr

Copy link
Copy Markdown
Contributor

Summary

A concise description of the problem being solved or the feature being added.

Type of Change

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Documentation update (README, CONTRIBUTING, comments)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Motivation & Context

Why is this change required? What problem does it solve? If it fixes an open issue, link it here (e.g. Fixes #123).

Testing Performed

  • I have run the test suite locally.
  • I have added tests that prove my fix is effective or that my feature works.

This PR improves the README installation and testing instructions.

  • Clarify the frontend installation steps.
  • Add comments explaining directory navigation.
  • Update the test coverage command after verifying the correct package path.

These changes make the setup instructions easier to follow for new contributors.

@Kaap10 Kaap10 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for your contribution. Reviewed and verified. The frontend setup and test coverage instructions are clearer and easier for contributors to follow.

@Kaap10
Kaap10 merged commit ee55976 into PicadoLabs:main Oct 29, 2026
4 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