diff --git a/.github/workflows/gradle.yml b/.github/workflows/gradle.yml index 561195c2..1deefcf2 100644 --- a/.github/workflows/gradle.yml +++ b/.github/workflows/gradle.yml @@ -3,7 +3,13 @@ name: Build mod -on: [push, pull_request] +on: + push: + branches: + - '**' + tags-ignore: + - '**' + pull_request: jobs: build: diff --git a/common/src/main/java/org/embeddedt/modernfix/dynamicresources/ModelBakeryHelpers.java b/common/src/main/java/org/embeddedt/modernfix/dynamicresources/ModelBakeryHelpers.java index ae72d6f0..7f33e184 100644 --- a/common/src/main/java/org/embeddedt/modernfix/dynamicresources/ModelBakeryHelpers.java +++ b/common/src/main/java/org/embeddedt/modernfix/dynamicresources/ModelBakeryHelpers.java @@ -104,21 +104,21 @@ public class ModelBakeryHelpers { } } allPackResources.removeIf(pack -> { - if(isTrustedPack.test(pack)) { - for(String namespace : pack.getNamespaces(PackType.CLIENT_RESOURCES)) { - Collection allBlockstates = pack.getResources(PackType.CLIENT_RESOURCES, namespace, "blockstates", p -> p.getPath().endsWith(".json")); - for(ResourceLocation blockstate : allBlockstates) { - allAvailableStates.add(new ResourceLocation(blockstate.getNamespace(), blockstate.getPath().replace("blockstates/", "").replace(".json", ""))); - } - Collection allModels = pack.getResources(PackType.CLIENT_RESOURCES, namespace, "models", p -> p.getPath().endsWith(".json")); - for(ResourceLocation blockstate : allModels) { - allAvailableModels.add(new ResourceLocation(blockstate.getNamespace(), blockstate.getPath().replace("models/", "").replace(".json", ""))); - } + for(String namespace : pack.getNamespaces(PackType.CLIENT_RESOURCES)) { + Collection allBlockstates = pack.getResources(PackType.CLIENT_RESOURCES, namespace, "blockstates", p -> p.getPath().endsWith(".json")); + for(ResourceLocation blockstate : allBlockstates) { + allAvailableStates.add(new ResourceLocation(blockstate.getNamespace(), blockstate.getPath().replace("blockstates/", "").replace(".json", ""))); + } + Collection allModels = pack.getResources(PackType.CLIENT_RESOURCES, namespace, "models", p -> p.getPath().endsWith(".json")); + for(ResourceLocation blockstate : allModels) { + allAvailableModels.add(new ResourceLocation(blockstate.getNamespace(), blockstate.getPath().replace("models/", "").replace(".json", ""))); } - return true; } - ModernFix.LOGGER.debug("Pack with class {} needs manual scan", pack.getClass().getName()); - return false; + if(!isTrustedPack.test(pack)) { + ModernFix.LOGGER.debug("Pack with class {} needs manual scan", pack.getClass().getName()); + return false; + } + return true; }); gatherAdditionalViaManualScan(allPackResources, allAvailableStates, blockStateFiles, "blockstates/"); @@ -206,6 +206,7 @@ public class ModelBakeryHelpers { blockStateData = null; blockStateLoadedFiles.clear(); + modelFiles.addAll(allAvailableModels); /* figure out which models we should actually load */ gatherAdditionalViaManualScan(allPackResources, allAvailableModels, modelFiles, "models/"); modelFiles.retainAll(allAvailableModels);