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

Cleanups for accepting scm-staging bot submissions #2980

Merged
merged 1 commit into from
Jun 29, 2023

Conversation

dirkmueller
Copy link
Member

  • Ignore .gitattributes in checkout
  • Skip Name: check when it is a scm-staging bot submission (relies on build service API that is broken for git packages)
  • Skip devel-group-maintainer check

check_source.py Outdated Show resolved Hide resolved
check_source.py Outdated Show resolved Hide resolved
check_source.py Outdated Show resolved Hide resolved
@dirkmueller dirkmueller force-pushed the bot_fixups branch 4 times, most recently from 1a6929f to a8a9d98 Compare June 28, 2023 22:49
check_source.py Outdated Show resolved Hide resolved
* Ignore .gitattributes in checkout
* Skip Name: check when it is a scm-staging bot submission
  (relies on build service API that is broken for git
  packages)
* Skip devel-group-maintainer check
self.review_messages['declined'] = "A package submitted as %s has to build as 'Name: %s' - found Name '%s'" % (
target_package, expected_name, new_info['name'])
return False
# TODO(dmllr): self.package_source_parse cannot handle scm_sync submissions, so skip the check for now
Copy link
Member

Choose a reason for hiding this comment

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

In which way does it fail?

Copy link
Member

@Vogtinator Vogtinator left a comment

Choose a reason for hiding this comment

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

Changes look good, just the commit title is wrong now - this is more than cleanups

@dirkmueller dirkmueller merged commit 3b612be into openSUSE:master Jun 29, 2023
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