Skip to content

feat: rebasing #216#464

Open
Droid00000 wants to merge 11 commits into
shardlab:mainfrom
Droid00000:feat/dakeuri-misc-changes
Open

feat: rebasing #216#464
Droid00000 wants to merge 11 commits into
shardlab:mainfrom
Droid00000:feat/dakeuri-misc-changes

Conversation

@Droid00000

Copy link
Copy Markdown
Collaborator

Summary

Rebases the PR to keep the one change which is still relevant. This is the last one for tonight

Dakurei and others added 11 commits June 15, 2024 10:20
  + Rspec tasks (for Ruby 2.7.x, 3.0.x, 3.1.x, 3.2.x)
  + Rubocop task (with Ruby 2.7.x)
  + Yard & Github pages tasks (with Ruby 2.7.x)

  + Update rspec version constraint in discordrb.gemspec (from ~> 3.11.0 to ~> 3.12.0)
  + Update redcarpet version constraint in discordrb.gemspec (from ~> 3.5.0 to ~> 3.6.0)
  Finally, after doing my tests on the latest version of Rspec, and looking at the code, I realize that the modification was not necessary, and PixeLInc was right in his PR shardlab#199
  Contrary to what the documentation indicated, it was not possible to provide an Array<Overwrite> to Channel#permission_overwrites=

  It was necessary to provide the Hash similar to what the method Channel#permission_overwrites returns

  + Added an example file to test the overwrites (I used it as a debug)
  + Modification of a test in spec/data/channel_spec.rb following this correction
Fix typo and newlines

Co-authored-by: Adrien Smith <adrien@panissupraomnia.dev>
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