Issue when Github user did not provide their full name, we would get None under reporter/comment user/etc. This PR attempts to fix that.
None
rebased onto 1bc43b15a9dccfe840a3daca4a32e9c68a1e44f1
FYI, you can do this in shorthand with 'author': comment.user.name or comment.user.login,
'author': comment.user.name or comment.user.login,
Same here.
One suggestion above, otherwise :+1:.
rebased onto e7feb7d3616984d48745e39113e2b13f1268e383
rebased onto 167d7ffd291e680ab4585bf91f06f5c77caf8991
153 new commits added
Remove converting markdown by default, now optional flag that needs to be placed in the updates array
Add comment when attempting to transition to CUSTOM_STATE
Adding converter to change github markdown to plaintext
Fixing cutstom labels downstream feature and updating README
Adding tests to cover new functions added in PR#97
Updating docs to reflect 'develop' param in config
Only send failure email if we are not developing
Add support to perform inital JIRA query to check status of JIRA server
Add 75% min coverage and added tests accordingly
Adding min coverage to tox
Updating downstream tests and some elements in main.py to be complient with flake8
Fixing small issues in qa-contact
Adding support for email to be sent with traceback in case of failure
Fixing change status doc string
Adding QA contact field support
Refactored status transition and updated README
Added configuration for custom ticket status
Adding check for JIRA issue fields before checking
Editing readme to link to documentation
Add ReadTheDocs documentation
Adding overwrite field to assignee
Removing hours from comments, and removing hours from status
Configuring tests to work with new commet format
Changing year to date in comment
Changing from utc epoch, to just from epoch
Add more accurate logs to update title
Adding regex and upstream reporter to title update
adding regex matches to all description updates
adding another catch for updating descrtiption
Adding jinja2 to DockerFile
Removing mailer class and replacing with static function
adding admin support for emails
Mailer gets config values from enviormental variables
Adding support to email owners in the situation that duplicate issues are created
Adding catch for PR comment
adding more tests for username and comment functions
Nevermind. Unnecessary.
Try fedora 29.
condenced function to find recently updated issue in matching
Removing hard-coded user in comment parsing, now pull from config dict
issue matching will always return a value now
adding appropriate test cases
Added title matching for Pagure
Fixing issue when ID is interpreted as String
Adding more checks for matching issues
added duplicate issue comment catching
Fixed issue where user has no updates array
PyGithub is a dep.
Reset the UPDATE_DATE.
Ignore xunit-tests.
changing xunit-tests from json to xml
Drop py35.
Bring back nose as a dep.
Make sure tox is available in the slave.
Fix up dependencies in the specfile.
No need to test on python2 any more.
Remove python2 sub-package. No longer needed.
requests is actually okay at any version.
Pull specfile from repo.
changing test case command in template
Add more sync features
Make sure to copy the specfile into the Dockerfile.
Added support for custom fields and custom ticket type for repository
Handle one more transition for RHV.
Handle this second exception.
Try closing issues with a specific resolution.
Slightly more flexible duplicate handling.
Apply filters for comment messages.
Github pull requests are on the same topic as issues. Distinguish!
Only synchronize issues when projects opt-in.
Simpler logging configuration.
Update default jira connection config to match code.
Refactor and simplify broken code block.
New log statement.
Flake8.
Remove unnecessary client.
Added tests for comments sync
Made jira and github/pagure comments intersection more understandable, return logging level to previous value and remove print
Bring mocks in line with latest comment sync changes.
PEP8 futzing.
Remove print statement.
Added support comment sync for pagure
Split line in _comment_matching
Add comments synchronization for Github
Remove pylint.
Add flake8.
downstream: Add ability to use different jira instances
Some notes from working with Bohdan Iakymets
Optionally listen to the message bus.
Remove old ansible config.
Need git.
Pass the argument all the way through.
Try setting this ENV explicitly.
Correct deps.
Correct path for pip3.
Verbose, please.
No need for this anymore.
Ensure command is present, and leave some other content around for debugging.
Pylint.
Try deleting the bc a different way.
Restore flake checks.
Get container build working.
Track latest fedora.
Fix xunit declaration.
Ignore rpmbuild output.
Use specfile for builddeps.
rpmbuild tools.
Pull in the spec file.
Use f28.
Install builddeps.
Add back missing variables for usernames to work.
nosetests
Version.sh.
Use older syntax to work with default jenkins master.
Revert "Restore docs building even thought it doesn't make sense for now."
Restore docs building even thought it doesn't make sense for now.
Only quay, thank you.
Messing with openshift.
Add in syncing for coreos/ignition
Add upstream fedora CoreOS repo syncing
Add "cl" label to issues imported from coreos/bugs
Handle various pagure message formats.
Typofix.
add syncing for openshift/os repo
Make "sst_coreosct" and "upstream" different labels
Rename estuary_updater to estuary-updater
Enable syncing issues for the Factory estuary_updater project
add upstream_cl and upstream_atomic labels
Atomic Teams now use COREOS project.
More labels For COREOS issue syncing
Component name: estuary.
Added the CoreOS bugs Repo and a JIRA label to use on this repo.
New bodhi milestone to track.
Set a component for finishline.
Enable syncing issues for ralphbean/finishline
Also, purview-api.
Add purview.
Remove botas.
Add botas.
Turn off BPO syncing.
Ignore deploy.retry.
Components for a few FACTORY projects.
Drop appstream.
Update JIRA label for fedora-ci/AtomicCi
sync2jira.py: Change github atomic label to sst_atomic_host
sync2jira.py: Change pagure atomic label to sst_atomic_host
Clean up pagure tag extraction.
Careful, now.
Seems that this needs to be status instead of state.
Add the upstream label back to atomic items.
Merge branch 'release/1.7'
rebased onto 8ada93e0a01771440362751a87fc6714b4571cdc
Accidently rebased on master :100: going to close this and open another PR, whoops
Pull-Request has been closed by sidpremkumar
Issue when Github user did not provide their full name, we would get
Noneunder reporter/comment user/etc. This PR attempts to fix that.