Skip to content

fix: remove index in DeleteDomains method#815

Merged
hsluoyz merged 1 commit intoapache:masterfrom
tangyang9464:master
Jun 11, 2021
Merged

fix: remove index in DeleteDomains method#815
hsluoyz merged 1 commit intoapache:masterfrom
tangyang9464:master

Conversation

@tangyang9464
Copy link
Contributor

@tangyang9464 tangyang9464 commented Jun 11, 2021

Signed-off-by: tangyang9464 tangyang9464@163.com
fix #814

@tangyang9464 tangyang9464 changed the title remove index in DeleteDomains method fix: remove index in DeleteDomains method Jun 11, 2021
Copy link
Contributor

@sagilio sagilio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Please fix:
image
You need add fix: in commit message and with -s option.

…domains.go

Signed-off-by: tangyang9464 <1410359230@qq.com>
@tangyang9464
Copy link
Contributor Author

@sagilio fixed

@hsluoyz hsluoyz merged commit aeeb9e6 into apache:master Jun 11, 2021
@github-actions
Copy link

🎉 This PR is included in version 2.31.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The index parameter is not used by the DeleteDomains(...) method

3 participants