Skip to content

Nitpicking: Remove comments - #7

Merged
entpiske merged 2 commits into
mainfrom
nitpicking
Jul 21, 2026
Merged

Nitpicking: Remove comments#7
entpiske merged 2 commits into
mainfrom
nitpicking

Conversation

@entpiske

Copy link
Copy Markdown
Member

No description provided.

@entpiske
entpiske requested a review from mhfs July 21, 2026 21:48
@entpiske
entpiske merged commit 64f88c8 into main Jul 21, 2026
18 checks passed
entpiske added a commit that referenced this pull request Jul 22, 2026
* fix(security): relax devise constraint to allow devise 5.x

Relaxes the devise dependency from `< 5` to `< 6` to allow
consumers to upgrade to devise 5.0.3 which fixes CVE-2026-32700
(confirmable race condition).

Tiddle uses only stable Devise APIs (Devise.add_module,
Devise::Strategies::Authenticatable, Devise.token_generator)
that are unchanged in devise 5.x.

Co-Authored-By: Claude Opus 4.6 <[email protected]>

* chore(ci): drop Rails 6.1 from test matrix

Rails 6.1 is EOL and incompatible with Ruby 3.x (LoggerThreadSafeLevel
errors). All consumers use Rails 7.2+.

Co-Authored-By: Claude Opus 4.6 <[email protected]>

* Bump dependencies (#5)

* bump dependencies

* remove not needed deps

* Add a changelog entry

* update GHA

* fix mongoid

---------

Co-authored-by: André Diego Piske <[email protected]>

* Simplify things (#6)

* Simplify things

* Adds support for `config.authentication_keys` in hash format

Co-authored-by: Arek W <[email protected]>

* remove appraisals

* database_cleaner not needed here explicitly

---------

Co-authored-by: André Diego Piske <[email protected]>
Co-authored-by: Arek W <[email protected]>

* Nitpicking: Remove comments (#7)

* remove comment

* there is no gemfile

---------

Co-authored-by: André Diego Piske <[email protected]>

---------

Co-authored-by: Mateus Bittencourt <[email protected]>
Co-authored-by: Claude Opus 4.6 <[email protected]>
Co-authored-by: Mateus Bittencourt <[email protected]>
Co-authored-by: André Diego Piske <[email protected]>
Co-authored-by: Arek W <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants