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.

Sign up for GitHub

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

Jump to bottom

Fix synthetic property #3848

Closed

andresrc wants to merge 1 commit into elastic:main from andresrc:synth-source

Collaborator

andresrc commented Jul 27, 2022

According to elastic/elasticsearch#85649 this is the way to set property.


          Fix synthetic property

60ffec7

andresrc requested review from joshdover and lalit-satapathy

July 27, 2022 06:47

andresrc requested a review from a team as a code owner

July 27, 2022 06:47

Collaborator Author

andresrc commented Jul 27, 2022

I did not see #3823 , closing

andresrc closed this

elasticmachine commented Jul 27, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-07-27T06:47:44.038+0000
Duration: 237 min 34 sec

Test stats 🧪

Test	Results
Failed	0
Passed	5351
Skipped	7
Total	5358

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

elasticmachine commented Jul 27, 2022

🌐 Coverage report

Name	Metrics % (`covered/total`)	Diff
Packages	100.0% (`280/280`)	💚
Files	97.126% (`507/522`)	👍
Classes	97.126% (`507/522`)	👍
Methods	89.26% (`4895/5484`)	👍
Lines	90.586% (`89163/98429`)	👍
Conditionals	100.0% (`0/0`)	💚

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

Labels

None yet