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 <noreply@anthropic.com>

* 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 <noreply@anthropic.com>

* Bump dependencies (#5)

* bump dependencies

* remove not needed deps

* Add a changelog entry

* update GHA

* fix mongoid

---------

Co-authored-by: André Diego Piske <andrepiske@gmail.com>

* Simplify things (#6)

* Simplify things

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

Co-authored-by: Arek W <arek01@gmail.com>

* remove appraisals

* database_cleaner not needed here explicitly

---------

Co-authored-by: André Diego Piske <andrepiske@gmail.com>
Co-authored-by: Arek W <arek01@gmail.com>

* Nitpicking: Remove comments (#7)

* remove comment

* there is no gemfile

---------

Co-authored-by: André Diego Piske <andrepiske@gmail.com>

---------

Co-authored-by: Mateus Bittencourt <mateus.bittencourt@trustedhealth.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Mateus Bittencourt <mateus.lbittencourt@gmail.com>
Co-authored-by: André Diego Piske <andrepiske@gmail.com>
Co-authored-by: Arek W <arek01@gmail.com>
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