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

md: Update BUILDING, CONTRIBUTING, PR-BUILD, ISSUE_TEMPLATE #10214

Merged
merged 4 commits into from
Apr 30, 2021

Conversation

AniLeo
Copy link
Member

@AniLeo AniLeo commented Apr 29, 2021

  • .github: Update CONTRIBUTING.md
  • .github: Update PR-BUILD.md
  • Update BUILDING.md
  • .github: Update issue templates
    Mention settings must be as close to defaults as possible and that game
    patches must be disabled when reporting issues

@AniLeo
Copy link
Member Author

AniLeo commented Apr 29, 2021

Images for PR-BUILD.md

image

image

image

@AniLeo AniLeo changed the title [WIP] md: Update BUILDING, CONTRIBUTING, PR-BUILD md: Update BUILDING, CONTRIBUTING, PR-BUILD Apr 29, 2021
@AniLeo AniLeo marked this pull request as ready for review April 29, 2021 23:50
@Megamouse
Copy link
Contributor

Mention no-patch policy?

AniLeo added 4 commits April 30, 2021 17:13
Modernize it a little bit
- Update Vulkan SDK references to match the version currently used in CI 
(1.2.154)
- Update Python to the oldest still supported version (3.6), also the 
version currently available on bionic
- Update leftover minimum required CMake version to 3.16.9
Mention settings must be as close to defaults as possible and that game 
patches must be disabled when reporting issues
@AniLeo AniLeo requested a review from Megamouse April 30, 2021 16:19
@AniLeo AniLeo changed the title md: Update BUILDING, CONTRIBUTING, PR-BUILD md: Update BUILDING, CONTRIBUTING, PR-BUILD, ISSUE_TEMPLATE Apr 30, 2021
@AniLeo AniLeo merged commit 2f44523 into RPCS3:master Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants