kb173 commented 3 years ago

Instructions are available here: https://docs.godotengine.org/en/latest/community/asset_library/submitting_to_assetlib.html

A .gitattributes file with export-ignore attributes like in https://github.com/aaronfranke/godot-measuring-tape/blob/master/.gitattributes should be added to ignore all the source files, as they are not relevant for using the pre-built addon.

We'll probably want to restructure the demo Godot project folders so that they are in the root directory. Maybe we should move the SConstruct script and the godot-cpp link into the src folder to keep all building-related stuff in there.

To do

[ ] #35 in order to provide a working demo with the asset
[ ] #42 so that there are no other external dependencies
[ ] #41 because manually building for all platforms all the time would be crazy
[ ] #51 to provide the full Geodot functionality on Mac

clemens-tolboom commented 3 years ago

.gitattributes is key.
addons/geodot should be rooted imho
I prefer the demo to be in addons/geodot/demo which can be ignored by the user importing
We are targetting Godot 3.2 so maybe stable https://docs.godotengine.org/en/stable/tutorials/assetlib/uploading_to_assetlib.html is better
I did not knew Plugin/Create tool

kb173 commented 3 years ago

Since we have build artifacts for Linux now (#41), I wonder if it'd make sense to upload these to a separate repository and use that for the asset library? https://github.com/cpina/github-action-push-to-another-repository

clemens-tolboom commented 3 years ago

I think it's enough to have the correct structure in this repo.

addons/geodot/gdnative/**
addons/geodot/demo/**
cpp/
.gitattributes (excluding all but addons)

Each release is a developer decision/testing/tagging which is still manual for the AssetLib is it?

(my2cents)

clemens-tolboom commented 3 years ago

With this setup we can add test scenes into the root and maybe use Godot testing with https://github.com/bitwes/Gut ?

clemens-tolboom commented 3 years ago

Comment move to Mac specific https://github.com/boku-ilen/geodot-plugin/issues/51#issuecomment-778651379

boku-ilen / geodot-plugin

Add Geodot to Godot's Asset Library #36

To do