-
Notifications
You must be signed in to change notification settings - Fork 554
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
[hi] Localize security-chaos-engineering.md #2608
Conversation
Signed-off-by: abhay-raj19 <rajpootabhay423@gmail.com>
✅ Deploy Preview for cncfglossary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm , what about others!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some suggestions!
commit if you find its useful 👍
Let's have other approvers suggestions as well
@bishal7679 now check up on those changes, have corrected them . |
Pls do review up the changes if you are satisfied with the changes do approve the request from your end @bishal7679 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nits: just remove the extra lines L 10, 16, 19, 26
## यह क्या है | ||
|
||
|
||
सुरक्षा अव्यवस्था इंजीनियरिंग(SCE) एक ऐसी शाखा है जो की [सीमावर्ती प्रणाली](/chaos-engineering/) पर आधारित है। |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
सुरक्षा अव्यवस्था इंजीनियरिंग(SCE) एक ऐसी शाखा है जो की [सीमावर्ती प्रणाली](/chaos-engineering/) पर आधारित है। | |
सुरक्षा अव्यवस्था इंजीनियरिंग(SCE) एक ऐसी शाखा है जो की [अव्यवस्था इंजीनियरिंग](/chaos-engineering/) पर आधारित है। |
@kumarankit999 @bishal7679 @aj11anuj @jayesh-srivastava , made the changes as suggested now review it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks fixed now, what other approvers say??
pls remove L23 and squash all the commits |
now fixed the suggested changes pls do check from your end @kumarankit999 @bishal7679 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @abhay-raj19 appreciate your effort!
Approver while merging make sure to apply squash and merge! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your efforts!
Signed-off-by : abhay-raj19 rajpootabhay423@gmail.com
Describe your changes
Localised a security chaos engineering document in hindi.
Ref: #2057
As this PR was being successfully reviewed by @jayesh-srivastava and was waiting for second Reviewer.
But due to some changes from my end this PR got deleted suddenly.
And re-open feature was not being available from my end .
Related issue number or link (ex:
resolves #issue-number
)#1986
Checklist before opening this PR (put
x
in the checkboxes)git commit -s
) is to affirm that commits comply DCO. If you are working locally, you could add an alias to yourgitconfig
by runninggit config --global alias.ci "commit -s"
.