Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix validationErrors type #361

Merged
merged 1 commit into from
Jul 22, 2023
Merged

Fix validationErrors type #361

merged 1 commit into from
Jul 22, 2023

Conversation

bc-m
Copy link
Contributor

@bc-m bc-m commented Jul 21, 2023

In the examples it is shown that the value can be read. Therefore, the value should not be marked as private.

@cjbarth
Copy link
Contributor

cjbarth commented Jul 22, 2023

Thank you for bringing this to our attention. There are actually a few typing errors like this. I'll review this and see if this fix by itself is sufficient.

@codecov
Copy link

codecov bot commented Jul 22, 2023

Codecov Report

Merging #361 (5378148) into master (0711039) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #361   +/-   ##
=======================================
  Coverage   72.19%   72.19%           
=======================================
  Files           9        9           
  Lines         856      856           
  Branches      227      227           
=======================================
  Hits          618      618           
  Misses        137      137           
  Partials      101      101           
Impacted Files Coverage Δ
src/signed-xml.ts 74.18% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cjbarth cjbarth merged commit af91bc2 into node-saml:master Jul 22, 2023
@cjbarth cjbarth added the bug label Jul 23, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants