Skip to content

docs: Removes obsolete QA document #543

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

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

docs: Removes obsolete QA document #543

wants to merge 1 commit into from

Conversation

tuliomir
Copy link
Contributor

Motivation

A QA document from before #139 was not relevant anymore and could be safely removed. Some additional clarification was added to the Readme file to explain its absence.

Acceptance Criteria

  • Removes the obsolete QA document

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@tuliomir tuliomir added the documentation Improvements or additions to documentation label Apr 29, 2025
@tuliomir tuliomir requested a review from r4mmer April 29, 2025 20:35
@tuliomir tuliomir self-assigned this Apr 29, 2025
@tuliomir tuliomir moved this from Todo to In Progress (Done) in Hathor Network Apr 29, 2025
Copy link

codecov bot commented Apr 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.42%. Comparing base (6f3d6e9) to head (0aa8d97).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #543   +/-   ##
=======================================
  Coverage   90.42%   90.42%           
=======================================
  Files          56       56           
  Lines        2422     2422           
  Branches      452      452           
=======================================
  Hits         2190     2190           
  Misses        216      216           
  Partials       16       16           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tuliomir tuliomir requested a review from Copilot April 29, 2025 22:47
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes an obsolete QA document and updates documentation references to reflect that removal.

  • Removed the QA document from the docs directory.
  • Updated the docs/README.md file to remove the QA link.
  • Updated the root README.md file to reference current automated testing guidelines instead of the removed QA test sequence.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
docs/README.md Removed the outdated QA link.
docs/QA.md Deleted the obsolete QA document.
README.md Updated test section: replaced QA test sequence with current guidelines.

@tuliomir tuliomir moved this from In Progress (Done) to In Review (WIP) in Hathor Network May 15, 2025
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 priority/low
Projects
Status: In Review (WIP)
Development

Successfully merging this pull request may close these issues.

2 participants