Skip to content

Skip flaky external-editor test in CI#1920

Merged
rolandwalker merged 1 commit into
mainfrom
RW/skip-flaky-external-editor-test
Jun 6, 2026
Merged

Skip flaky external-editor test in CI#1920
rolandwalker merged 1 commit into
mainfrom
RW/skip-flaky-external-editor-test

Conversation

@rolandwalker
Copy link
Copy Markdown
Contributor

@rolandwalker rolandwalker commented Jun 6, 2026

Description

This test often fails, which is unhelpful.

Incidentally move an item in the changelog.

I don't know the behave suite very well, but hope this works based on the Python 3.12 example.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@rolandwalker rolandwalker self-assigned this Jun 6, 2026
@rolandwalker rolandwalker force-pushed the RW/skip-flaky-external-editor-test branch from c284917 to 72eb951 Compare June 6, 2026 12:25
@rolandwalker rolandwalker requested review from amjith and scottnemes June 6, 2026 12:30
@rolandwalker
Copy link
Copy Markdown
Contributor Author

44 scenarios passed, 0 failed, 0 skipped

Maybe not working yet.

@rolandwalker rolandwalker force-pushed the RW/skip-flaky-external-editor-test branch from 72eb951 to 2cc2fd1 Compare June 6, 2026 12:41
This test often fails, which is unhelpful.

Incidentally move an item in the changelog.
@rolandwalker rolandwalker force-pushed the RW/skip-flaky-external-editor-test branch from 2cc2fd1 to d1223b1 Compare June 6, 2026 12:50
@rolandwalker
Copy link
Copy Markdown
Contributor Author

Working:

43 scenarios passed, 0 failed, 1 skipped

in https://github.com/dbcli/mycli/actions/runs/27062778536/job/79878345269?pr=1920

@rolandwalker rolandwalker merged commit 78dddea into main Jun 6, 2026
11 checks passed
@rolandwalker rolandwalker deleted the RW/skip-flaky-external-editor-test branch June 6, 2026 18:04
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