Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

docs: Improved parsing of upstream branch name#1328

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@bartdorlandt
Copy link
Contributor

@bartdorlandtbartdorlandt commentedSep 11, 2025
edited
Loading

When using renovate it may present branch names with additional "." (dots) in the branch name. This will break the upstream branch lookup. This updated version focuses on the 3 consecutive dots instead.

Purpose

Improve docs example on github CI pipeline, specifically on locating the upstream branch.

Rationale

When using renovate it may present branch names with additional "." (dots) in the branch name. This will break the upstream branch lookup. This updated version focuses on the 3 consecutive dots instead and selects the upstream branch name.

How did you test?

I changed the code in my own pipeline.
https://github.com/bartdorlandt/convert_poetry2uv/pull/44/files

How to Verify

Execute the old and new command on upstream branches, one with a "." in the name and one without.


PR Completion Checklist

note: the link doesn't function anymore.

  • Changes Implemented & Validation pipeline succeeds

  • Commits follow theConventional Commits standard
    and are separated into the proper commit type and scope (recommended order: test, build, feat/fix, docs)

  • Appropriate Unit tests added/updated

  • Appropriate End-to-End tests added/updated

  • Appropriate Documentation added/updated and syntax validated for sphinx build (see Contributor Guidelines)

@bartdorlandtbartdorlandtforce-pushed thedocs_upstream_branch branch 2 times, most recently from9c50d09 to40e6311CompareSeptember 11, 2025 13:05
@codejedi365
Copy link
Contributor

Thanks for the recommendation and fix

bartdorlandt reacted with thumbs up emoji

@codejedi365codejedi365 merged commitccc91c0 intopython-semantic-release:masterSep 13, 2025
8 checks passed
@codejedi365
Copy link
Contributor

🎉 This PR has been published as part of v10.4.1 🎉

You can find more information about this release on theGitHub Releases page.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bartdorlandt@codejedi365

[8]ページ先頭

©2009-2025 Movatter.jp