Skip to content
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

Remove un-used renameSlot method #2286

Merged
merged 3 commits into from Apr 16, 2023
Merged

Remove un-used renameSlot method #2286

merged 3 commits into from Apr 16, 2023

Conversation

bibhu107
Copy link
Contributor

@bibhu107 bibhu107 commented Apr 16, 2023

What is the purpose of this pull request?

  • Documentation update
  • Bug fix
  • Feature addition or enhancement
  • Code maintenance
  • DevOps
  • Improve developer experience
  • Others, please explain:

Fixes #2284

Overview of changes:

Anything you'd like to highlight/discuss:

Testing instructions:

Proposed commit message: (wrap lines at 72 characters)
Remove un-used renameSlot method


Checklist: ☑️

  • Updated the documentation for feature additions and enhancements
  • Added tests for bug fixes or features
  • Linked all related issues
  • No unrelated changes

@tlylt tlylt requested a review from jovyntls April 16, 2023 07:00
@jovyntls
Copy link
Contributor

jovyntls commented Apr 16, 2023

Hi @bibhu107 , thank you for contributing to Markbind! 🥳

We usually try to fix one issue per PR to avoid unrelated changes and keep a trackable history. Would you mind ensuring that this PR fixes only one issue? You could tackle another issue (not tagged "good first issue") if you're feeling up to it - we try to have one "good first issue" per contributor to encourage more new contributors :) If you'd like to fix more than one, feel free to open second PR!

Would be nice if you could fill in the template given in the issue description as well, especially the proposed commit message! (To check a checkbox, you can fill it in with an 'x', e.g. [x])

@bibhu107
Copy link
Contributor Author

Hello @jovyntls ,
Thanks for the suggestions! I have updated the PR description and made sure this PR has fix for only issue #2284 .

Thanks!

@jovyntls jovyntls changed the title Fixing issue #2284 Removing un-used method renameSlot Apr 16, 2023
Copy link
Contributor

@jovyntls jovyntls left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks again @bibhu107 🎉

@jovyntls jovyntls added this to the v4.1.1 milestone Apr 16, 2023
@jovyntls jovyntls changed the title Removing un-used method renameSlot Remove un-used renameSlot method Apr 16, 2023
@jovyntls jovyntls merged commit 5490de8 into MarkBind:master Apr 16, 2023
5 checks passed
@jovyntls
Copy link
Contributor

@all-contributors please add @bibhu107 for code

@allcontributors
Copy link
Contributor

@jovyntls

I've put up a pull request to add @bibhu107! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove unused renameSlot method
2 participants