Skip to content

[DO NOT MERGE][ci] Do not prompt for authentication on macos - #23306

Draft
dpiparo wants to merge 1 commit into
root-project:masterfrom
dpiparo:testSolutionGitMac
Draft

[DO NOT MERGE][ci] Do not prompt for authentication on macos#23306
dpiparo wants to merge 1 commit into
root-project:masterfrom
dpiparo:testSolutionGitMac

Conversation

@dpiparo

@dpiparo dpiparo commented Sep 8, 2026

Copy link
Copy Markdown
Member

This Pull request:

Changes or fixes:

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

This PR fixes #

else:
branch = ""

terminal_prompt_off = "GIT_TERMINAL_PROMPT=0" if is_macos() else ""

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does it matter if we also do this on other platform? (i.e. is there anywhere where we would want to be prompted for password?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

thanks for the remark. this is an admittedly half-crazy attempt to try and debug the problem... On Windows that would not have worked for example. On Linux everything works. Using an env variable injected like that is in any case not something I would recommend

@dpiparo dpiparo changed the title [ci] Do not prompt for authentication on macos [DO NOT MERGE][ci] Do not prompt for authentication on macos Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 12h 14m 22s ⏱️
 3 883 tests  3 883 ✅ 0 💤 0 ❌
76 160 runs  76 160 ✅ 0 💤 0 ❌

Results for commit 43fe6c8.

♻️ This comment has been updated with latest results.

@dpiparo dpiparo closed this Sep 8, 2026
@dpiparo dpiparo reopened this Sep 8, 2026
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.

3 participants