• Made phabricator lister robust

@anlambert As you recommended I tested the lister on multiple forges. Some of the repos where it failed to fetch URL are -

• Updated cli and made phabricator more robust

@anlambert I will test my lister on various phabricator instance and fix the bug. Also, I update the readme according to your recommendation. Thanks for your feedback.

• Fixed a typo

• Updated cli and readme for phabricator lister

• Made test cases for priority url selector

• Made improvements in code quality

Thanks, @vlorentz and @anlambert. I will implement these changes and submit the diff ASAP.
I have one more doubt. I do not have much experience in writing test cases. Can you please advice me or recommend me some source where I can refer to(maybe some lister which is already implemented), to get the idea on writing test cases to validate the repository URL extraction approach.

• Added priority url selector in phabricator lister

@anlambert I have made all the recommended changes in the code. Could you please review it once?

• Made phabricator listor robust

Thanks, @anlambert for your reply. I am facing one more issue. How should I decide the priority order of the URIs. Is there any important point which I should keep into consideration while deciding the priority order?

@anlambert I am not able to understand the difference between raw uri, display uri, effective uri and normalized uri, can you please help me with this and explain a bit about the difference between these 4 types of uri?

Thank you @anlambert for your in-depth review. Thanks for testing the code. I will surely update my code according to your revision by earliest. Thanks again for such detailed comments. These help me a lot in solving the problem.

@ardumont @anlambert Can you please review the code and suggest improvements

• Fixed test task of phabricator lister

• Changed indexable value in phabricator lister

• fixed conversion of datatype

• Fixed null exception in shortName

• Changed indexable and uid type to string

• Changed bad response for phabricator lister

Hey @faux, I am still working on this.

Thanks, @ardumont, I will surely use print statements and tox to debug this code. I have my end semester exam coming hence cannot devote much time on working on this. Although I will try my best to complete this in a couple of weeks. I hope this is fine with you.

• Fixed spelling mistake in phabricator lister

Respected all,
I am now to this community and this is my first task.Can you please mentor me in solving this?

Resected all,
I would love to complete this task. And it would be a good starting point for me for implementing new listers.

Thanks for the feedback.

Shall I change the commit message and then merge?

Oh, sorry.

The sample top levels for the different listers are also out of date(as discussed in IRC as well as mentioned by @ardumont in https://forge.softwareheritage.org/D1281). Hence I wish to update the names of those functions in the documentation of lister.

@ardumont Ok, from next time I will remove unnecessary commits and rebase interactively. And I will go through the Git_style_guide once again. Thanks for your feedback. I will surely remember this for future references.

• Removed Extra blank space

• Removed extra space from the README
• Removed unnecessary files

Ok, next time I will remember this to paste the text and quote it with triple backquote "`".
I would love to fix the documentation as it would be a good starting point in contributing to this project. I am familiar with the process of pull request on Github but new to this forge. Can you guide to on what should I do next?