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

Add 6 games; Precise contributing docs, especially "opens" flag requirements; fix typos #59

Closed
wants to merge 0 commits into from

Conversation

mattmaniak
Copy link
Member

@mattmaniak mattmaniak commented Jun 24, 2024

My proposal is targeted to clarify the opens flag which says whether a game "opens"... The issue I have found with it is that it was not clear if a game that crashes after 1, 5 or 10 seconds can be considered as "opening". Now a requirement to mark a game as "opening" is ability to launch it to it's default screen (the one that is loaded after launch without using any input) which is most often a main menu.

I altered games mostly added by me with an updated opens flag also.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
src/game-support/README.md Outdated Show resolved Hide resolved
src/game-support/call-of-cthulhu-dcote.md Outdated Show resolved Hide resolved
src/game-support/coj-bound-in-blood.md Outdated Show resolved Hide resolved
src/game-support/splinter-cell.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
src/game-support/sw-battlefront2-2005.md Outdated Show resolved Hide resolved
src/game-support/sw-battlefront2-2005.md Outdated Show resolved Hide resolved
src/game-support/sniper-gw.md Outdated Show resolved Hide resolved
@mattmaniak
Copy link
Member Author

@JoshuaBrest all code review proposals are applied.

@mattmaniak
Copy link
Member Author

Golden Just Cause 2 has been added.

@mattmaniak mattmaniak changed the title Add 3 games; Precise contributing docs, especially "opens" flag requirements, fix typos Add 4 games; Precise contributing docs, especially "opens" flag requirements; fix typos Jul 7, 2024
@mattmaniak mattmaniak changed the title Add 4 games; Precise contributing docs, especially "opens" flag requirements; fix typos Add 5 games; Precise contributing docs, especially "opens" flag requirements; fix typos Jul 7, 2024
@mattmaniak
Copy link
Member Author

Add Deus Ex: Human Revolution - Director's Cut.

@mattmaniak
Copy link
Member Author

mattmaniak commented Jul 7, 2024

Add Kholat title - silver.

@mattmaniak mattmaniak changed the title Add 5 games; Precise contributing docs, especially "opens" flag requirements; fix typos Add 6 games; Precise contributing docs, especially "opens" flag requirements; fix typos Jul 7, 2024
src/game-support/README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@JoshuaBrest JoshuaBrest left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request game addition New game addition to the docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants