morucci / repoxplorer

RepoXplorer - Git stats explorer
Apache License 2.0
116 stars 24 forks source link

Added functional --all-branches option. #64

Closed evangineer closed 3 years ago

evangineer commented 4 years ago

Include all branches on repos with this option.

Satisfies #62

morucci commented 3 years ago

Hi, I've tested you code. The yaml generated file looks good to me. I think it can be merged but first could you explain a bit more what are you seeing as issues ? Could you give me a way to reproduce the issue ?

evangineer commented 3 years ago

Focusing on something else right now, but I suspect the problem is with the indexer not properly handling the multiple branches.

morucci commented 3 years ago

I've merged your changes. See https://softwarefactory-project.io/r/19761. This repository is just a mirror of the official repo on softwarefactory-project.io. https://github.com/morucci/repoxplorer/commit/be40c7aba7b0b15a200cbc8c5c3ac23cc06ba779

For the issue you mentioned, I'll have a deeper look. That's weird because this is tested in the tests suite here https://github.com/morucci/repoxplorer/blob/master/repoxplorer/tests/test_indexer_git.py#L584

morucci commented 3 years ago

Thanks !