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

insert semicolon in Lclass;method for mixin target #64

Merged
merged 1 commit into from
Jul 21, 2021

Conversation

spaceclouds42
Copy link
Contributor

Fixes an issue where Mixin targets for methods would not have the ; between class and method descriptors

@boring-cyborg
Copy link

boring-cyborg bot commented Jul 21, 2021

Hello, and thanks for opening a pull request! As this is your first pull request to this repository we'd just like to offer you a warm welcome to the project, and the following pointers:

  • Most importantly, all contributions must adhere to our Code of Conduct and contributor guidelines. Please give them a quick read if you haven't already.
  • While our team is passionate about the projects we've created here, we're all volunteers. Please don't be offended if it takes time for us to get to your pull request - we'll be here as soon as we can be!
  • If your pull request is a work-in-progress, then please mark it as a draft (if you haven't already done so). This helps us to prioritise our review process.
  • Our review process may feel a little daunting to a newcomer. Don't worry, everyone starts somewhere! In order to make things easier, please ensure that you're linting your code. You can find more information on how to do this in the README for this project.
    Thanks again for your work so far! If you have any questions, please feel free to join us on Discord in the #dev-kotdis channel or to contact a staff member directly.

@gdude2002
Copy link
Member

Yaaaaaaaay, thanks again haha

@gdude2002 gdude2002 merged commit b44a93c into Kord-Extensions:develop Jul 21, 2021
@boring-cyborg
Copy link

boring-cyborg bot commented Jul 21, 2021

Congratulations, you've just scored your first merged pull request to this repository! Thanks for your contribution - your changes are currently being built and will be deployed automatically.
Kotlin Discord UG exists to support its users, and it's people like you that help to shape our path as we move forward. We couldn't do what we do without your support!

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.

2 participants