#105 Refactor TracImporter
Closed by cverna. Opened by cverna.
cverna/pagure-importer refactor_importer_object  into  master

Download 105.patch

Removed the TracImporter object, so that we can use the trac importer function as a library.

Now that I moved most of the API calls to a function per call. I can create more unit test that I will push in a later commit.

rebased

This isn't done automatically?

/me likes :)

2 new commits added

  • Simple test for create_comment function
  • Remove TracImporter object from test suite

4 new commits added

  • Remove TracImporter object from test suite
  • Remove TracImporter object and use function instead
  • Move logic to import issues into fedorahosted command code
  • Beginning of refactor to remove TracImporter object

4 new commits added

  • Remove TracImporter object from test suite
  • Remove TracImporter object and use function instead
  • Move logic to import issues into fedorahosted command code
  • Beginning of refactor to remove TracImporter object

I need the an empty list, if not set it will default to None and break later :confounded:

Looks like it needs a rebase :)

rebased

rebased

Closing this PR to merge into a different branch than master

Pull-Request has been closed by cverna

Metadata