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

NAFF V1 #488

Merged
merged 64 commits into from
May 24, 2022
Merged

NAFF V1 #488

merged 64 commits into from
May 24, 2022

Conversation

LordOfPolls
Copy link
Member

What type of pull request is this?

  • Non-breaking code change
  • Breaking code change
  • Documentation change/addition
  • Tests change

Description

Goodbye snek, hello NAFF. This PR concludes our rebrand from Dis-Snek to NAFF, and as an extra treat; Improves parts of the library/repo.

Note to org-member reviewers:
By approving this PR you are indicating you have no breaking changes planned short term. Obviously we don't want to release NAFF 2.0.0 a day after 1.0.0

Changes

  • Renames Snake client -> Client
  • Renames Scale to Cog
  • Renames models/snek -> models/naff
  • Renames SnakeException -> NaffException
  • Renames SnakeBotUser -> NaffUSer
  • Renames grow_scale -> mount_cog
  • Renames shed_scale -> drop_cog
  • Renames get_scales -> Client.get_cogs
  • Renames get_scale -> Client.get_cogs
  • Renames Snake.sync_scales -> Client.sync_cogs
  • Renames Snake.scales -> Clientcogs
  • Renames Command.scale -> Command.cog
  • Renames Scale.*scale* -> Cog.*cog*
  • Renames raw_socket_event -> raw_gateway_event
    Probably a load more renames
  • Add automated testing bot
  • Performance Improvements
  • Add auto-mod support
  • Application.cover_image is now an Asset
  • Expose Application attrs for App Discovery
  • Move FormData processing into http_client.request

Checklist

  • I've formatted my code with Black
  • I've ensured my code works on Python 3.10.x
  • I've tested my code

LordOfPolls and others added 30 commits May 2, 2022 19:18
Consts are... constant, use a tuple
* tests: Merge snek-tests into lib pytests

* test: install voice deps to env

* tests: skip voice on non-voice extra

* fix: Wait for member update event

* tests: Use pyproj for coverage

* tests: Remove now redundant pragmas

* tests: prevent multiple test-bots running concurrently
@LordOfPolls
Copy link
Member Author

Re-requesting reviews due to changes

@LordOfPolls
Copy link
Member Author

LordOfPolls commented May 23, 2022

For transparency - it is my intent to merge this PR as soon as the currently active PRs have merged, and this pr is approved by 2 contributors afterwards.

Pending PRs:
#484 - if tested by its author
#499 - Merged
#500 - Merged
#501 - Merged

* fix: Generate fallback channel object when Forbidden

* fix: prevent caching errors with pre-cached recipients

* fix: generate dummy DM channel on demand
* fix: Ensure subprocess is killed on linux

* fix: handle small audio files better
Copy link
Contributor

@zevaryx zevaryx left a comment

Choose a reason for hiding this comment

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

Am using this on my production bot without issues. I think it's ready to go

@LordOfPolls LordOfPolls merged commit f5e2b30 into master May 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants