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

added setup info for PiCade #23

Closed
wants to merge 1 commit into from
Closed

Conversation

rdmueller
Copy link
Contributor

just a proposal. Feel free to edit it.

@Vegz78
Copy link
Owner

Vegz78 commented Sep 2, 2021

Hi @rdmueller,

Thanks for the PR! Even though I appreciate the proposal and the chance to learn how to follow through a PR from this side of the repo ownership, I feel the proposed text, specific only for the Picade is out of place inside the main readme.md document.

Ideally, since broad controller support is crucial for the usefulness of McAirpos, I maybe should've had a wiki dedicated to the various tested controllers, and moved some information from the main readme.md.

However, since more documents also means more follow-up work, I have chosen to use the issues section as a wiki/knowledge base instead, with mention of the relevant solutions for the various popular and tested controllers high up in the readm.md, like this:
image

If you could be so kind to as add your PR tekst as a concluding summary for the Picade at the end of your issue instead, I will add a similar line for the Picade in the readme.md, with attribution for your help and link to this issue/summary.

@Vegz78 Vegz78 closed this Sep 2, 2021
@Vegz78 Vegz78 reopened this Sep 2, 2021
@Vegz78 Vegz78 closed this Sep 2, 2021
@Vegz78 Vegz78 added documentation Improvements or additions to documentation Controller Controller related issue labels Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Controller Controller related issue documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Picade] Button mapping(working with manual /sd/arcade.cfg edit and nomap option)
2 participants