Skip to content

feat: NC34 support#192

Merged
oleksandr-nc merged 1 commit intomainfrom
nc34-support
Apr 24, 2026
Merged

feat: NC34 support#192
oleksandr-nc merged 1 commit intomainfrom
nc34-support

Conversation

@oleksandr-nc
Copy link
Copy Markdown
Contributor

@oleksandr-nc oleksandr-nc commented Apr 21, 2026

Why the tests were changed:

After a successful TOTP login during OAuth authorize, GitHub now sometimes serves a Verify your two-factor authentication (2FA) settings checkup interstitial with a /settings/two_factor_checkup/delay ("remind me later") form.

@oleksandr-nc oleksandr-nc requested a review from kyteinsky as a code owner April 21, 2026 13:05
@oleksandr-nc oleksandr-nc force-pushed the nc34-support branch 2 times, most recently from 3e5394a to 74d8253 Compare April 21, 2026 16:31
The NC34 max-version bump was already on main but never released.
This bumps the version, adds the changelog entry, and bumps
league/commonmark to 2.8.2 to clear the roave/security-advisories
conflict that was breaking the static-psalm-analysis CI jobs.

Signed-off-by: Oleksander Piskun <oleksandr2088@icloud.com>
Copy link
Copy Markdown
Member

@julien-nc julien-nc left a comment

Choose a reason for hiding this comment

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

Nice test adjustment

@oleksandr-nc oleksandr-nc merged commit b283f12 into main Apr 24, 2026
30 checks passed
@oleksandr-nc oleksandr-nc deleted the nc34-support branch April 24, 2026 08:02
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