Skip to content

Space age update#75

Merged
Danielv123 merged 22 commits intomasterfrom
space-age-update
Feb 25, 2025
Merged

Space age update#75
Danielv123 merged 22 commits intomasterfrom
space-age-update

Conversation

@Danielv123
Copy link
Member

@Danielv123 Danielv123 commented Dec 8, 2024

Adds compatibility with space age while maintaining compatibility with 0.17, 1.0 and 1.1

The control stage now uses the new compat function from clusterio_lib. I changed the versioning scheme from 1.99.???+1/2 to 2.minor.factorio_version. This makes it a lot easier to tell which version belongs to which factorio version.

@Danielv123 Danielv123 marked this pull request as ready for review January 24, 2025 18:33
@Danielv123
Copy link
Member Author

Almost forgot - need to add locale for the default fluid recipes before merge.
I would also like the new graphics from hurricane unless that holds things up too much

Danielv123 and others added 3 commits January 25, 2025 16:08
Co-authored-by: HurricaneFUE5 <rzeifart@gmail.com>
Co-authored-by: HurricaneFUE5 <rzeifart@gmail.com>
Co-authored-by: HurricaneFUE5 <rzeifart@gmail.com>
@Danielv123
Copy link
Member Author

image

build.js Outdated
Comment on lines +274 to +276
(async () => {
await main().catch(err => { console.log(err) });
})();
Copy link
Member

Choose a reason for hiding this comment

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

What does this async warp achieve?

Copy link
Member Author

Choose a reason for hiding this comment

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

Remnant from extra tooling I run in my local environment, i could revert it but the diff is cleaner like this

@@ -1,9 +1,8 @@
-- Auto generated by post.py, do not edit
Copy link
Member

Choose a reason for hiding this comment

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

Is this file no longer generated by post.py? I do not see any changes to that file.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, blender render is gone, I should remove it.

@Cooldude2606
Copy link
Member

Two issues found by manual testing.

  1. Fluid selection recipes do not have the correct locale name. This may be a result of 2.0 changing the resolution method for recipe names.
  2. Fluid selection recipes include blueprint parameters. These need to be filtered out using either parameter == false or auto_barrel == true

@Danielv123 Danielv123 merged commit 18da205 into master Feb 25, 2025
2 checks passed
@Danielv123 Danielv123 deleted the space-age-update branch February 25, 2025 17:00
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