Skip to content
This repository has been archived by the owner on May 16, 2022. It is now read-only.

closing issues using ogr #231

Merged
merged 5 commits into from
Jul 16, 2019
Merged

Conversation

marusinm
Copy link
Collaborator

Related to #190, #193, but still not fully implement these issues.

I replaced Github API with ogr for closing and checking Issues.
Integration tests are passing for Github, unit tests are provided in ogr. I still need to add several features into ogr to fully support Pagure, then I can write tests for Pagure too.

Copy link
Contributor

@rpitonak rpitonak left a comment

Choose a reason for hiding this comment

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

Nice changes!

I left a few comments, my concern here is that user should be able to figure out what is wrong, without checking the logs of RB - it means just through GH interface.

Edit: Sorry, I approved by mistake, should be Changes are requested

release_bot/configuration.py Show resolved Hide resolved
release_bot/releasebot.py Outdated Show resolved Hide resolved
Copy link
Member

@jpopelka jpopelka left a comment

Choose a reason for hiding this comment

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

LGTM with one suggestion

release_bot/releasebot.py Outdated Show resolved Hide resolved
@jpopelka jpopelka merged commit c495449 into user-cont:master Jul 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants