-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
Installing v3.0.0 from Asset Library tries to add the whole repository #29
Comments
Apparently it can be configured using
|
Should be okay now. Documentation and examples or ignored when exported from Github |
Thanks. Just a minor issue: LICENSE.md is still exported. I noticed it's duplicate with the one in add-ons, but maybe you want to keep a copy at the root so it applies to examples too, and GitHub detects it properly. In this case, you could just add it to export-ignore. |
I also realize the examples were quite useful, we'll have to think about a way to restore them in a clean way. Either we keep them at the root and do not export-ignore them, but we add some sub-folder In the meantime, I'll just clone the repo and copy the examples. |
OK, the issue you see just above (Examples bad zoom) has a discussion containing an argument against importing the examples by default (they may not work out of the box with certain game window settings). I'll try to discuss this further with the Godot team, as allowing standalone examples would be great. EDIT: I opened a discussion if you're curious: godotengine/godot-proposals#6546 |
When installing v3.0.0 from the Asset Library, I noticed that all files and folders can be imported:
It's possible to uncheck some of the file/folders, but even then they will be placed widly in project root. It would be nice if files not needed in other projects were ignored, and others rearranged under addons/ or alternatively some sub-folder with a more specific name.
Note that I don't know how other add-ons like GUT manage to ignore files and folders. I've heard of .gdignore but GUT doesn't seem to use it, and it would only work for folders.
The text was updated successfully, but these errors were encountered: