website icon indicating copy to clipboard operation
website copied to clipboard

improve Enum documentation

Open techmannih opened this issue 1 year ago • 7 comments

What kind of change does this PR introduce? improve Enum documentation Issue Number:

  • Closes #521

techmannih avatar Jan 14 '25 12:01 techmannih

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
website ✅ Ready (View Log) Visit Preview cc0acf7bdb9015e29d0d7c6dd5054f1a0fea0327

github-actions[bot] avatar Jan 14 '25 12:01 github-actions[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (219521e) to head (cc0acf7). Report is 57 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1312   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines          396       396           
  Branches       106       106           
=========================================
  Hits           396       396           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

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

codecov[bot] avatar Jan 14 '25 12:01 codecov[bot]

@benjagm @valeriahhdez review this PR

techmannih avatar Jan 15 '25 09:01 techmannih

@techmannih pls. attach some screenshots and screencasts showing your changes

DhairyaMajmudar avatar Jan 31 '25 18:01 DhairyaMajmudar

@DhairyaMajmudar sure,

https://github.com/user-attachments/assets/0259b344-f1ff-4f35-bdaa-8c5781d49b18

techmannih avatar Feb 01 '25 03:02 techmannih

@benjagm please review this now.

techmannih avatar Mar 04 '25 17:03 techmannih

@benjagm @gregsdennis please review this now.

techmannih avatar Mar 22 '25 15:03 techmannih

I still think the 42 in there is weird since it doesn't fit with the "traffic light" application, but it works as an example.

gregsdennis avatar Jun 14 '25 07:06 gregsdennis