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

fix doc type of AceInlineScreenReader to enable API generation #5355

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

InspiredGuy
Copy link
Contributor

API generation failed during 1.30.0 release due to import("../editor") statement, removing it to enable API generation for next releases. API docs were generated already with this statement removed.

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

@InspiredGuy InspiredGuy requested a review from akoreman October 13, 2023 16:12
@codecov
Copy link

codecov bot commented Oct 13, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (8e41ac8) 87.46% compared to head (9a0a34c) 87.46%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5355   +/-   ##
=======================================
  Coverage   87.46%   87.46%           
=======================================
  Files         577      577           
  Lines       45858    45858           
  Branches     6954     6954           
=======================================
  Hits        40109    40109           
  Misses       5749     5749           
Flag Coverage Δ
unittests 87.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/autocomplete/inline_screenreader.js 100.00% <ø> (ø)

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

@InspiredGuy InspiredGuy merged commit 3086776 into master Oct 13, 2023
3 checks passed
@InspiredGuy InspiredGuy deleted the fix-doc-type branch October 13, 2023 16:20
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