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

build: updatable snapshots #27

Merged
merged 4 commits into from
Feb 24, 2024
Merged

build: updatable snapshots #27

merged 4 commits into from
Feb 24, 2024

Conversation

noomorph
Copy link
Contributor

@noomorph noomorph commented Feb 24, 2024

Scope of work

  • No actual code has been changed, only tests and scripts.

  • Based test snapshots keys on IDs, not on Interslavic notation (to produce cleaner diffs)
    image

  • Added npm run import to update fixtures automatically from Google Sheets
    image

  • Added rich test reports (Allure) to preview and discuss changes in the conjugation algorithms

telegram-cloud-photo-size-2-5390972547291533753-y

@coveralls
Copy link

coveralls commented Feb 24, 2024

Pull Request Test Coverage Report for Build 8030303191

Details

  • 0 of 27 (100.0%) changed or added relevant lines in 2 files are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.05%) to 94.929%

Files with Coverage Reduction New Missed Lines %
src/verb/conjugationVerb.ts 2 89.77%
Totals Coverage Status
Change from base Build 7956424675: -0.05%
Covered Lines: 1554
Relevant Lines: 1629

💛 - Coveralls

Copy link
Contributor

github-actions bot commented Feb 24, 2024

View the deployed report at surge.sh

@noomorph noomorph merged commit 933ba89 into main Feb 24, 2024
1 check passed
@noomorph noomorph deleted the build/updatable-snapshots branch February 24, 2024 11:52
Copy link
Contributor

github-actions bot commented Mar 2, 2024

🎉 This PR is included in version 2.3.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants