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

Add message_link/channel_link to MessageReference #1877

Merged
merged 10 commits into from Apr 14, 2024

Conversation

Le0Developer
Copy link
Contributor

Summary

Adds the make_link() function of Message to MessageReference.

Checklist

  • I have run nox and all the pipelines have passed. (my setup is borked)
  • I have made unittests according to the code I have added/modified/deleted.

Copy link
Contributor

@beagold beagold left a comment

Choose a reason for hiding this comment

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

Thanks for the pull request! Left some comments

changes/1877.feature.md Outdated Show resolved Hide resolved
hikari/messages.py Outdated Show resolved Hide resolved
hikari/messages.py Outdated Show resolved Hide resolved
hikari/messages.py Outdated Show resolved Hide resolved
@beagold
Copy link
Contributor

beagold commented Apr 12, 2024

Test failures are unrelated and will be fixed by #1881. I will later merge that over this pull request and I believe it should be good to merge.

Thanks again!

hikari/messages.py Outdated Show resolved Hide resolved
Le0Developer and others added 9 commits April 14, 2024 10:17
Co-authored-by: beagold <86345081+beagold@users.noreply.github.com>
Signed-off-by: Leo Developer <LeoDeveloper@protonmail.com>
Co-authored-by: beagold <86345081+beagold@users.noreply.github.com>
Signed-off-by: Leo Developer <LeoDeveloper@protonmail.com>
beagold
beagold previously approved these changes Apr 14, 2024
@beagold
Copy link
Contributor

beagold commented Apr 14, 2024

@Le0Developer running nox -s black should fix the linting issues, after which I think this is good to merge

@Le0Developer Le0Developer changed the title Add make_link() to MessageReference Add message_link/channel_link to MessageReference Apr 14, 2024
@beagold beagold merged commit 9a7d58a into hikari-py:master Apr 14, 2024
27 checks passed
@Le0Developer Le0Developer deleted the feat/messageref-makelink branch April 14, 2024 08:51
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.

None yet

3 participants