Skip to content

Make list_contributors.sh target ref configurable#16364

Open
r0qs wants to merge 1 commit intodevelopfrom
inform-remote-on-list-contributors-script
Open

Make list_contributors.sh target ref configurable#16364
r0qs wants to merge 1 commit intodevelopfrom
inform-remote-on-list-contributors-script

Conversation

@r0qs
Copy link
Member

@r0qs r0qs commented Dec 18, 2025

The script previously hardcoded origin/develop, which fails when the upstream repository is configured under a different remote name. This PR fixes that and adds optional second parameter to specify the target ref, defaulting to the local develop branch.

@r0qs r0qs requested a review from nikola-matic December 18, 2025 17:44
@r0qs r0qs self-assigned this Dec 18, 2025
@r0qs r0qs requested a review from cameel December 19, 2025 19:46
cameel
cameel previously approved these changes Dec 20, 2025
@cameel cameel force-pushed the inform-remote-on-list-contributors-script branch from 4e63898 to dd688be Compare December 20, 2025 00:32
@r0qs r0qs force-pushed the inform-remote-on-list-contributors-script branch from dd688be to 430ff4b Compare December 20, 2025 03:23
@github-actions github-actions bot added the stale The issue/PR was marked as stale because it has been open for too long. label Jan 3, 2026
@argotorg argotorg deleted a comment from github-actions bot Jan 5, 2026
@r0qs r0qs removed the stale The issue/PR was marked as stale because it has been open for too long. label Jan 5, 2026
@github-actions github-actions bot added the stale The issue/PR was marked as stale because it has been open for too long. label Jan 20, 2026
@clonker clonker removed the stale The issue/PR was marked as stale because it has been open for too long. label Jan 20, 2026
@argotorg argotorg deleted a comment from github-actions bot Jan 26, 2026
@r0qs r0qs force-pushed the inform-remote-on-list-contributors-script branch from 430ff4b to 9790f26 Compare January 26, 2026 18:44
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