Skip to content

fix(types): correct TypeCast's Next callback to return unknown#3129

Merged
wellwelwel merged 3 commits into
sidorares:masterfrom
bbugh:next-type-unknown
Oct 31, 2024
Merged

fix(types): correct TypeCast's Next callback to return unknown#3129
wellwelwel merged 3 commits into
sidorares:masterfrom
bbugh:next-type-unknown

Conversation

@bbugh

@bbugh bbugh commented Oct 17, 2024

Copy link
Copy Markdown
Contributor

Fixes "typeCast Next type appears to be incorrect" #3122

Also replaced references to the old incorrect type signature in referencing comments

Fixes typeCast Next type appears to be incorrect sidorares#3122
@codecov

codecov Bot commented Oct 17, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.13%. Comparing base (b92db2a) to head (955de6f).
Report is 27 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3129   +/-   ##
=======================================
  Coverage   88.13%   88.13%           
=======================================
  Files          71       71           
  Lines       12890    12890           
  Branches     1355     1355           
=======================================
  Hits        11361    11361           
  Misses       1529     1529           
Flag Coverage Δ
compression-0 88.13% <ø> (ø)
compression-1 87.33% <ø> (-0.80%) ⬇️
tls-0 87.55% <ø> (ø)
tls-1 87.89% <ø> (ø)

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

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

@wellwelwel wellwelwel 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.

Thanks, @bbugh.

I've suggested some rollback changes to the website's test files, or if you prefer, the CI error is due to the last EOL character in ./website/test/fixtures/external-code-embed/QueryOptions.txt 🤝

Comment thread website/test/fixtures/external-code-embed/QueryOptions.txt Outdated
Comment thread website/test/resources/external-code-embed/random-methods.txt Outdated
@wellwelwel wellwelwel linked an issue Oct 18, 2024 that may be closed by this pull request
@bbugh

bbugh commented Oct 31, 2024

Copy link
Copy Markdown
Contributor Author

@wellwelwel I reverted the fixture files, it appears to be passing the checks now. Sorry for the trouble!

@wellwelwel

Copy link
Copy Markdown
Collaborator

Thanks again, @bbugh 🙋🏻‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

typeCast Next type appears to be incorrect

2 participants