Skip to content

Conversation

@j0eybrinkman
Copy link
Contributor

@j0eybrinkman j0eybrinkman commented Jan 17, 2024

Removed out-dated info. php 8 is now included in the AppStream repo.

Author checklist (Completed by original Author)

  • [ x] Good fit for the Rocky Linux project? Title and Author Metatags inserted ?
  • [ x] If applicable, steps and instructions have been tested to work
  • [ x] Initial self-review to fix basic typos and grammar completed

Rocky Documentation checklist (Completed by Rocky team)

  • 1st Pass (Document is good fit for project and author checklist completed)
  • 2nd Pass (Technical Review - check for technical correctness)
  • 3rd Pass (Detailed Editorial Review and Peer Review)
  • Final approval (Final Review)

This note is no longer useful now that php 8 is in the AppStream repos.
@github-actions
Copy link

Test results for 47b5315:

Number of broken URLs: 3

URL,RESULT,FILENAME
 https://www.yourdomain.com/,failed,guides/security/generating_ssl_keys_lets_encrypt.md
 https://www.ssllabs.com/ssltest/,failed,guides/web/caddy.md
 https://coding.tools/regex-tester,failed,books/admin_guide/15-three-swordsmen.md

@sspencerwire
Copy link
Contributor

sspencerwire commented Jan 17, 2024

@j0eybrinkman You aren't wrong. PHP 8.0 is currently available in Rocky Linux 8.9 as you note. However there is other content here that needs to change too. Thank you for this! I'll merge and then make the other changes.

@sspencerwire sspencerwire merged commit 2c01010 into rocky-linux:main Jan 17, 2024
@j0eybrinkman
Copy link
Contributor Author

Ok thank you, this is my first commit to a repo other than my own so figured i'd start small!

@j0eybrinkman j0eybrinkman deleted the patch-1 branch January 17, 2024 21:21
@sspencerwire
Copy link
Contributor

Ok thank you, this is my first commit to a repo other than my own so figured i'd start small!

I very much appreciate your contribution!!

@sspencerwire
Copy link
Contributor

@all-contributors please add @j0eybrinkman for content

1 similar comment
@sspencerwire
Copy link
Contributor

@all-contributors please add @j0eybrinkman for content

@allcontributors
Copy link
Contributor

@sspencerwire

I've put up a pull request to add @j0eybrinkman! 🎉

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