Skip to content

Conversation

benblank
Copy link
Contributor

While developing #37, I found it very helpful to have a Godot project set up for the addon. Not only does this allow the addon to be easily opened in the Godot editor, it also allows external editors (e.g. Visual Studio Code) to use Godot as a language server.

The project file is configured with compatibility with Godot 4.0 (so that Importality's supported versions of Godot don't change) and the .godot cache directory is ignored by Git.

@nklbdev
Copy link
Owner

nklbdev commented Nov 18, 2024

Thanks again! I'll check and add your code tomorrow!
The main thing is that there are no conflicts with the client project code when installing the plugin.

@benblank
Copy link
Contributor Author

The main thing is that there are no conflicts with the client project code when installing the plugin.

Oh, right. 🤦

I'm not sure how the plugin packaging works, but I already get conflicts when installing the current version from the Asset Library (e.g. README.md and icon.svg) so I didn't really make the connection. But I imagine conflicting with project.godot is potentially much worse! Or maybe just not possible? It definitely warrants testing.

@nklbdev
Copy link
Owner

nklbdev commented Nov 19, 2024

When installing a plugin from Godot Asset Library, a user can choose which files to install and avoid conflict.
I'll check everything, don't worry!

@benblank
Copy link
Contributor Author

benblank commented Dec 4, 2024

I've been rolling this around in the back of my head and I think the best option might be to introduce a "packaged" distributable for the addon. That way, the files at the top level of the project could be whatever they need to be, but would be removed (or relocated) during packaging.

Would you be interested in a PR for a GitHub Actions workflow which does that? I figure it could automatically trigger when you tag a new release and would create a zip file containing only the addons directory, but with the LICENSE* and README* files moved into addons/nklbdev.importality. That way, when users install the addon from the Asset Store or by downloading the package and expanding it into their project, they wouldn't have to worry about conflicts at all.

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