Skip to content

Conversation

@valencik
Copy link
Member

This PR consolidates all the project data to one yaml files as done in #274 but without the categorization, which could perhaps come later in a separate PR.

It also makes sure cats comes first in the list of cats-... named projects.

Screen Shot 2023-01-24 at 9 44 19 PM

I've kept changes to a minimum here, but I feel like this would make it easier to maintain reasonable category values across all the projects. Or ensure that all projects have particular metadata.

@valencik valencik self-assigned this Jan 25, 2023
@jducoeur
Copy link
Member

Mixed feelings, at first glance. Purely from a scalability POV, I kind of prefer having them separate, rather than in a single big file. But I may well be overly precious there -- in practice, it's probably fine. And if this allows us better control over the order of the listings, it may well be the right decision.

@valencik
Copy link
Member Author

Just a quick note to self, if we go with this approach we'll want to update the migration template: https://github.com/typelevel/governance/blame/main/.github/ISSUE_TEMPLATE/2-project-migration.md#L22

@zmccoy
Copy link
Member

zmccoy commented Jan 25, 2023

I think the single file is fine just ease of ordering etc. We can just keep an eye on if this becomes a pain.

Copy link
Contributor

@samspills samspills left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also a fan of the single file, and yah not impossible to chunk back up into individual files if we want that in the future

@valencik valencik merged commit 649d26f into development Jan 26, 2023
@valencik valencik deleted the proj-yml branch January 26, 2023 00:51
valencik added a commit to typelevel/governance that referenced this pull request Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants