Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

fix: #154 and #155 #156

Merged
merged 2 commits into from
Jan 4, 2022
Merged

fix: #154 and #155 #156

merged 2 commits into from
Jan 4, 2022

Conversation

genzyy
Copy link
Contributor

@genzyy genzyy commented Jan 4, 2022

add whitespace to whitelist and fix use of undefined variable

Description

  • Added whitespace to whitelist so the bot doesn't give warning messages.
  • Fixed use of undefined variable customMessage and use warningMessage instead which is defined and should have been used.

Check List (mark all applicable checkboxes)

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots (if applicable)

Original Updated
original screenshot updated screenshot

add whitespace to whitelist and fix use of undefined variable
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Yeah! You did it 🎉 Now, Relax 😉 -> Grab a drink ☕ -> And wait for the maintainers to check your contributions. Meanwhile, you can discuss on other issues and solve them 😀

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord Server - we help and encourage each other to contribute to open source little and often 🤓 Any questions, let us know!

Copy link
Contributor

@siddhant-khisty siddhant-khisty left a comment

Choose a reason for hiding this comment

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

Looks good. I just found another word that can be added.
Please add the word ques as well.

@genzyy
Copy link
Contributor Author

genzyy commented Jan 4, 2022

Looks good. I just found another word that can be added. Please add the word ques as well.

Sure!

@genzyy
Copy link
Contributor Author

genzyy commented Jan 4, 2022

Whitelisted ques.

Copy link
Contributor

@siddhant-khisty siddhant-khisty left a comment

Choose a reason for hiding this comment

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

LGTM

@siddhant-khisty siddhant-khisty merged commit 5433523 into WeMakeDevs:main Jan 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] use of undefined variable [BUG] Trigger for Whitespace
3 participants