Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modified/expanded some categories #3

Merged
merged 3 commits into from
Sep 24, 2020
Merged

Modified/expanded some categories #3

merged 3 commits into from
Sep 24, 2020

Conversation

rodrigomelo9
Copy link
Contributor

Related to #1

Doubts:

  • edalize and fusesoc are in the category "Tools:Package Managers"? In my opinion, edalize is a "Tools"Project Managers" or "Tools:Tool Managers". In case of fusesoc, it would be "Cores:Managers".
  • If there is a "Conferences" category, what about "Boards"?

@rodrigomelo9
Copy link
Contributor Author

P&R seems not a valid name into config.yaml. "Tools:Place And Route"? "Tools:Placer"?

@eine
Copy link
Contributor

eine commented Sep 24, 2020

As said in #1, I think it's ok to keep edalize and fusesoc in the same category, at least for now.

Yes, I think that a "Boards" category would be nice.

We can use "PnR" instead of "P&R", which is also commonly used (nextpnr, arachne-pnr...).

Also, added the Boards category and the list was alphabetically ordered.
@eine
Copy link
Contributor

eine commented Sep 24, 2020

The issue is with "Tools"Programmers". It should be "Tools:Programmers".

@eine
Copy link
Contributor

eine commented Sep 24, 2020

Nice! It's ready for merging, isn't it?

@rodrigomelo9
Copy link
Contributor Author

I think that it is ready. When needed we can add more categories.

@rodrigomelo9 rodrigomelo9 marked this pull request as ready for review September 24, 2020 03:40
@eine eine merged commit 1efe582 into hdl:develop Sep 24, 2020
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.

2 participants