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.

Sign up for GitHub

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

Jump to bottom

Add Fedora 39 #822

Merged
merged 1 commit into from
Nov 12, 2023
Merged

Add Fedora 39 #822

merged 1 commit into from
Nov 12, 2023

Conversation

Jai-JAP
Copy link
Contributor

@Jai-JAP Jai-JAP commented Oct 25, 2023 •
edited
Loading

  • Add exception for Fedora Onyx ( new in fedora 39 )

flexiondotorg
flexiondotorg requested changes Nov 9, 2023
View reviewed changes
Copy link
Member

@flexiondotorg flexiondotorg left a comment

Choose a reason for hiding this comment

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

Please update this pull request to only make the changes required to add support for Fedora 39.

@Jai-JAP
Copy link
Contributor Author

Jai-JAP commented Nov 9, 2023

Please update this pull request to only make the changes required to add support for Fedora 39.

Done

@flexiondotorg
Copy link
Member

This patch is massive.

image

Please refactor this pull request to only provide the changes to add support for Fedora 39 and remove all whitespace-related changes.

@flexiondotorg flexiondotorg marked this pull request as draft November 10, 2023 10:30
@flexiondotorg flexiondotorg force-pushed the master branch 5 times, most recently from 716c252 to 99e7677 Compare November 10, 2023 16:11
@Jai-JAP
Copy link
Contributor Author

Jai-JAP commented Nov 10, 2023 •
edited
Loading

https://getfedora.org/releases.json has removed all versions prior to fedora 38
Should I also remove all obsolete versions?

@flexiondotorg flexiondotorg marked this pull request as ready for review November 12, 2023 17:19
@flexiondotorg flexiondotorg merged commit f217e63 into quickemu-project:master Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philclifford philclifford philclifford approved these changes

@flexiondotorg flexiondotorg flexiondotorg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Jai-JAP @flexiondotorg @philclifford