Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

skip_changelog: Fix role repo discovery #461

Merged
merged 1 commit into from
Nov 18, 2024
Merged

Conversation

SuperQ
Copy link
Contributor

@SuperQ SuperQ commented Nov 17, 2024

Fix syntax of split to correctly select list item.

  • Add some extra stderr debugging logs.
  • Skip the _common role.
  • Skip roles where the repo is missing.

@SuperQ
Copy link
Contributor Author

SuperQ commented Nov 17, 2024

This will fix the updater cron job.

@SuperQ SuperQ changed the title Fix role repo discovery skip_changelog: Fix role repo discovery Nov 17, 2024
Fix syntax of split to correctly select list item.
* Add some extra stderr debugging logs.
* Skip the `_common` role.
* Skip roles where the repo is missing.

Signed-off-by: SuperQ <superq@gmail.com>
Copy link
Member

@gardar gardar left a comment

Choose a reason for hiding this comment

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

Nice!

@gardar gardar merged commit ef4eb15 into main Nov 18, 2024
23 checks passed
@gardar gardar deleted the superq/fix_discover_role branch November 18, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants