Handle default_branch and topics in Repository

[abestel]

No description provided.

[abestel]

Looking in https://github.com/github/rest-api-description it is safe to assume that default_branch is required and always returned.
Topics is optional, and I thought it was nicer to expose a List.empty[String] instead of a Option[List[String]] in the domain object.

[juanpedromoreno]

Thanks @abestel , would we need to update also the docs?

[abestel]

Thanks @abestel , would we need to update also the docs?

Thanks for checking my PR @juanpedromoreno :-)
I checked the documentation before submitting this change, and I don't think there's anything to update. The page for the repositories related actions is here and the model is referenced by linking directly to the actual source code, ie here.

I don't really understand why the Github Worflow failed by the way

[juanpedromoreno]

The pipeline failed for an unrelated issue, no worries about that, I checked your PR and everything looks right.

[juanpedromoreno]

question: Would it make sense to add the new fields at the end? Thinking about an easier way to upgrade for people using the existing model :)

[abestel]

I like to keep the required fields before the fields having a default value (hence the default_branch here and the topics all the way down) but I can definitely change it if you think it would be better :-)

[juanpedromoreno]

Your point makes sense to me too, let's keep it that way 👍

[juanpedromoreno]

Thanks so much for your time @abestel !